From 1132298e7c0e3e4c114d6d4388a617f9e93d0e4e Mon Sep 17 00:00:00 2001
From: Jerome Guelfucci <jeromeg@xfce.org>
Date: Fri, 5 Jun 2009 13:52:35 +0000
Subject: [PATCH] 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.



(Old svn revision: 7461)
---
 ChangeLog                   | 8 ++++++++
 lib/screenshooter-actions.c | 9 ++-------
 2 files changed, 10 insertions(+), 7 deletions(-)

diff --git a/ChangeLog b/ChangeLog
index 24240a42..2d1f5a5b 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -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.
diff --git a/lib/screenshooter-actions.c b/lib/screenshooter-actions.c
index ba10cfe9..40ba990c 100644
--- a/lib/screenshooter-actions.c
+++ b/lib/screenshooter-actions.c
@@ -69,16 +69,11 @@ gboolean screenshooter_take_and_output_screenshot (ScreenshotData *sd)
       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;
         }
     }
-- 
GitLab