[et-mgmt-tools] [PATCH] virt-manager: remove disambiguity in "free space" dialog

Cole Robinson crobinso at redhat.com
Tue Feb 17 00:51:02 UTC 2009


Guido Günther wrote:
> On Mon, Feb 09, 2009 at 08:21:29AM +0000, Mark McLoughlin wrote:
>> Hi Guido,
>>
>> On Sat, 2009-02-07 at 21:34 +0100, Guido Günther wrote:
>>> -                res = self.err.yes_no(_("Not Enough Free Space"), ret[1])
>>> +                res = self.err.yes_no(_("Not Enough Free Space"), _("%s. Continue anyway?") % ret[1])
>>>                  if not res:
>>>                      return False
>> See the GNOME HIG:
>>
>>   http://library.gnome.org/devel/hig-book/stable/windows-alert.html.en#alert-button-order
>>
>> I think what you want here is to change the buttons to Cancel/Continue -
>> i.e. a Cancel button and an affirmative verb button.
> I've changed this to "Ok/Cancel" in the attached patch. A simple Ok
> looks more obvious to me in this case since I'd be unsure to continue
> with what (it's not creating wizard in the case of add_hardware) and the
> only thing the user has to do is to acknowledge.
>  -- Guido
> 

Thanks, applied now:

http://hg.et.redhat.com/cgi-bin/hg-virt.cgi/applications/virt-manager--devel/rev/60940c2af2db

- Cole




More information about the et-mgmt-tools mailing list