Bug 475926 - Delete confirmation dialog nitpick
Summary: Delete confirmation dialog nitpick
Keywords:
Status: CLOSED UPSTREAM
Alias: None
Product: Virtualization Tools
Classification: Community
Component: virt-manager
Version: unspecified
Hardware: All
OS: Linux
low
low
Target Milestone: ---
Assignee: Daniel Berrangé
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2008-12-11 05:18 UTC by Daniel Bass
Modified: 2010-03-16 17:16 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2009-01-24 19:52:33 UTC
Embargoed:


Attachments (Terms of Use)

Description Daniel Bass 2008-12-11 05:18:23 UTC
User-Agent:       Mozilla/4.0 (compatible; MSIE 7.0; Windows NT 5.1; .NET CLR 1.1.4322; .NET CLR 2.0.50727)

The delete confirmation dialog says, 'This will permanently delete the vm "$vm," are you sure?' How about this instead: 'Are you sure you want to delete the virtual machine "$vm"?' It seems that this would be helpful in mentally parsing that dialog.
Also, the buttons in the dialog are "Yes" and "No". They should be "Delete" and "Cancel", as per the GNOME HIG: http://library.gnome.org/devel/hig-book/stable/windows-alert.html.en


Reproducible: Always

Comment 1 Richard Laager 2009-01-24 00:42:24 UTC
The text is addressed here: http://hg.et.redhat.com/cgi-bin/hg-virt.cgi/applications/virt-manager--devel/rev/e2056d9074e5

Cole Robinson had this to say about the buttons:
> This can hold off a bit. We need a revamped delete dialog anyways to
> optionally allow deleting the VM disk storage, so I'll incorporate this
> change when that work is done.

Comment 2 Cole Robinson 2009-01-24 19:52:33 UTC
Yes, there is already a bug to track the enhanced delete dialog, bug 351681. We can close this as UPSTREAM, and I'll add the button recommendation to the other bug.


Note You need to log in before you can comment on or make changes to this bug.