[virt-tools-list] [virt-viewer 2/2] Make 'Cancel' the default action in exit dialog

Christophe Fergeau cfergeau at redhat.com
Tue Oct 15 15:30:05 UTC 2013


On Tue, Oct 15, 2013 at 11:23:36AM -0400, Marc-André Lureau wrote:
> ack
> 
> ----- Original Message -----
> > In the 'Do you want to close the session dialog?', the default focus
> > is currently on the 'Do not ask me again' checkbox.
> > The purpose of this dialog is to make sure that the user does not
> > inadvertantly exit remote-viewer, this commit changes the default
> > action in this dialog to be 'cancel' rather than switching the
> > 'Do not ask me again 'checkbox.
> > ---
> >  src/virt-viewer-app.c | 1 +
> >  1 file changed, 1 insertion(+)
> > 
> > diff --git a/src/virt-viewer-app.c b/src/virt-viewer-app.c
> > index 445fe26..a272caf 100644
> > --- a/src/virt-viewer-app.c
> > +++ b/src/virt-viewer-app.c
> > @@ -297,6 +297,7 @@ virt_viewer_app_maybe_quit(VirtViewerApp *self,
> > VirtViewerWindow *window)
> >          gtk_container_add(GTK_CONTAINER(gtk_dialog_get_content_area(GTK_DIALOG(dialog))),
> >          check);
> >          gtk_widget_show(check);
> >  
> > +        gtk_dialog_set_default_response (GTK_DIALOG(dialog),
> > GTK_RESPONSE_OK);

Would have been better if I hadn't forgotten to amend that commit.
The actual patch has GTK_RESPONSE_CANCEL here instead of GTK_RESPONSE_OK.

Christophe
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/virt-tools-list/attachments/20131015/c3c8d18a/attachment.sig>


More information about the virt-tools-list mailing list