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 519360 - Add confirmation on exit
Add confirmation on exit
Status: RESOLVED OBSOLETE
Product: dasher
Classification: Applications
Component: general
unspecified
Other All
: Normal enhancement
: ---
Assigned To: Kevin Kubasik
Phil Cowans
Depends on:
Blocks: 519278 519313
 
 
Reported: 2008-02-28 18:44 UTC by Phil Cowans
Modified: 2021-05-26 09:58 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Dasher Exit Dialog (996 bytes, patch)
2008-04-22 01:50 UTC, Kevin Kubasik
none Details | Review
Updated with config option (2.62 KB, patch)
2008-05-01 22:48 UTC, Kevin Kubasik
none Details | Review

Description Phil Cowans 2008-02-28 18:44:30 UTC
Add a confirmation dialogue on exit even if no data needs to be saved (or Dasher is in a mode where it does not have an internal document). This should be configurable as per #519357.
Comment 1 Kevin Kubasik 2008-04-22 01:23:33 UTC
Hey, if no ones working on this, I'd like to give it a shot, seems like a good way to wet might feet with dasher.
Comment 2 Kevin Kubasik 2008-04-22 01:50:57 UTC
Created attachment 109666 [details] [review]
Dasher Exit Dialog

This adds a simple exit prompt whenever dasher is about to exit, not just when the file has changed. Probably needs some formatting/translation love. (I've never used gtkmm before and added 3 new strings)
Comment 3 Phil Cowans 2008-04-22 20:05:35 UTC
Thanks - I've applied the patch to trunk and confirmed that Dasher compiles, although I'm currently connecting over SSH so can't test it out properly.

Would you be willing to take a look at #519357? I'd suggest adding a boolean parameter to Src/Common/AppSettingsHeader.h and Src/Common/AppSettingsData.h and making this dialogue box appear only when that is set. Let me know if you need any more info, as the configuration options code can be a little difficult to work with at times.
Comment 4 Kevin Kubasik 2008-04-25 01:06:24 UTC
I'll look at it, but might need some help ;) 
Comment 5 Kevin Kubasik 2008-05-01 22:48:24 UTC
Created attachment 110250 [details] [review]
Updated with config option

Utilizes a config option, defaults to true.
Comment 6 Willie Walker 2008-12-08 15:47:00 UTC
Hi All: 

What's the status of this bug?  Does the attached patch resolve it?  If so, can we get it checked in?

Will
Comment 7 Sam 2009-02-22 19:01:51 UTC
Yes. Does the attached patch resolve it?

Thank you
Comment 8 Kevin Kubasik 2009-03-18 02:49:57 UTC
Hey! I don't know if this patch is much good anymore (its pretty old). It would be a great help if someone could try applying it against trunk and share the results. 
Comment 9 Kevin Kubasik 2009-03-18 02:50:03 UTC
Hey! I don't know if this patch is much good anymore (its pretty old). It would be a great help if someone could try applying it against trunk and share the results. 
Comment 10 André Klapper 2021-05-26 09:58:12 UTC
GNOME is going to shut down bugzilla.gnome.org in favor of gitlab.gnome.org.
As part of that, we are mass-closing older open tickets in bugzilla.gnome.org
which have not seen updates for a longer time (resources are unfortunately
quite limited so not every ticket can get handled).

If you can still reproduce the situation described in this ticket in a recent
and supported software version, then please follow
  https://wiki.gnome.org/GettingInTouch/BugReportingGuidelines
and create a new enhancement request ticket at
  https://gitlab.gnome.org/GNOME/dasher/-/issues/

Thank you for your understanding and your help.