[Goodies-commits] r7461 - in xfce4-screenshooter/trunk: . lib

[Jerome Guelfucci]

Author: jeromeg
Date: 2009-06-05 13:52:35 +0000 (Fri, 05 Jun 2009)
New Revision: 7461

Modified:
   xfce4-screenshooter/trunk/ChangeLog
   xfce4-screenshooter/trunk/lib/screenshooter-actions.c
Log:
Try to fix bug #5424.

Destroy the main dialog before doing anything else. Normally, this will
prevent this dialog from remaining displayed on the screen and being
captured instead of the active window.



Modified: xfce4-screenshooter/trunk/ChangeLog
===================================================================
--- xfce4-screenshooter/trunk/ChangeLog	2009-06-04 21:04:58 UTC (rev 7460)
+++ xfce4-screenshooter/trunk/ChangeLog	2009-06-05 13:52:35 UTC (rev 7461)
@@ -1,3 +1,11 @@
+2009-06-05 jeromeg
+
+Try to fix bug #5424.
+
+Destroy the main dialog before doing anything else. Normally, this will
+prevent this dialog from remaining displayed on the screen and being
+captured instead of the active window.
+
 2009-06-03 jeromeg
 
 Add the autotools stuff for the Turkish translation.

Modified: xfce4-screenshooter/trunk/lib/screenshooter-actions.c
===================================================================
--- xfce4-screenshooter/trunk/lib/screenshooter-actions.c	2009-06-04 21:04:58 UTC (rev 7460)
+++ xfce4-screenshooter/trunk/lib/screenshooter-actions.c	2009-06-05 13:52:35 UTC (rev 7461)
@@ -69,16 +69,11 @@
       g_signal_connect (dialog, "response", (GCallback) cb_help_response, NULL);
 
       response = gtk_dialog_run (GTK_DIALOG (dialog));
+      gtk_widget_destroy (dialog);
 
-      if (response == GTK_RESPONSE_OK)
+      if (response != GTK_RESPONSE_OK)
         {
-          gtk_widget_hide (dialog);
-        }
-      else
-        {
-          gtk_widget_destroy (dialog);
           gtk_main_quit ();
-
           return FALSE;
         }
     }