After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 573635 - Exit without save - asked twice for integrated UI case
Exit without save - asked twice for integrated UI case
Status: RESOLVED FIXED
Product: dia
Classification: Other
Component: general
0.97-pre2
Other Linux
: Normal normal
: ---
Assigned To: Dia maintainers
Dia maintainers
Depends on:
Blocks:
 
 
Reported: 2009-03-01 14:50 UTC by Hans Breuer
Modified: 2009-03-29 19:34 UTC
See Also:
GNOME target: ---
GNOME version: ---



Description Hans Breuer 2009-03-01 14:50:19 UTC
- start with integrted ui
- draw sowemthing
- choose exit
- hit "Exit without save"
The dialog appears once while the UI is visible and a second time when it is gone.
Comment 1 Hans Breuer 2009-03-28 11:35:47 UTC
a quich hack to overcome this error below. But it probably would be better to do all this "exit code" handling in exit_dialog rather than complicating app_procs.c even more. (If it the exit dialog would have a better visual appearance, it could be used for all ui cases.)

Index: app/app_procs.c
===================================================================
--- app/app_procs.c	(revision 4346)
+++ app/app_procs.c	(working copy)
@@ -973,6 +973,18 @@
           g_free (filename);
         }
         exit_dialog_free_items (items);
+      } 
+      else if (result == EXIT_DIALOG_EXIT_NO_SAVE) 
+      {
+        list = dia_open_diagrams();
+        while (list) {
+          diagram = list->data;
+
+	  /* slight hack: don't ask again */
+          diagram_set_modified (diagram, FALSE);
+	  undo_clear(diagram->undo);
+          list = g_list_next (list);
+	}
       }
     }
     else
 
Comment 2 Hans Breuer 2009-03-29 19:34:28 UTC
apparently there is not coming a better version in time, so I commited the above:

2009-03-29  Hans Breuer  <hans@breuer.org>
	
	* app/app_procs.c : slightly hacked version to avoid double request
	of "Exit without save" for integrated UI case, bug #573635