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 622181 - [backup-restore] Backup settings uses unhelpful yes/no dialog
[backup-restore] Backup settings uses unhelpful yes/no dialog
Status: RESOLVED FIXED
Product: evolution
Classification: Applications
Component: Plugins
2.28.x (obsolete)
Other Linux
: Normal normal
: ---
Assigned To: evolution-plugin-maintainers
Evolution QA team
Depends on:
Blocks:
 
 
Reported: 2010-06-20 12:04 UTC by coffeetastesawesome
Modified: 2010-07-29 06:49 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Patch to change the text on both dialogs (2.38 KB, patch)
2010-07-28 08:09 UTC, rugby471
none Details | Review
Patch that works this time (2.66 KB, patch)
2010-07-28 08:13 UTC, rugby471
committed Details | Review

Description coffeetastesawesome 2010-06-20 12:04:23 UTC
In latest Ubuntu Lucid (YAY!) with Evolution 2.28.3 (2.28.3-0ubuntu9), if I go to backup my settings (File->Backup settings), I am presented with a File Save dialog (which is just fine), and then a dialog box warning me that I should close Evolution.
This dialog, however, is an unfriendly and unhelpful yes/no dialog; further, the text of the dialog does not even make it clear which button to select to close Evolution.
This dialog should probably have two buttons - one reading, "Cancel", and the other, "Close Evolution".
Comment 2 rugby471 2010-07-28 08:09:29 UTC
Okay, here is a patch which changes both the restore and backup dialogs.

I have changed the backup dialog text from this:
---
Evolution backup can start only when Evolution is not running. Please make sure that you save and close all your unsaved windows before proceeding.  If you want Evolution to restart automatically after backup, please enable the toggle button.
---

To this:
---
To backup your data and settings, you must first close Evolution. Please make sure that you save any unsaved data before proceeding.
---

I removed the unnecessary explanation about the togglebutton, it is not needed, it can explain itself. I also removed the bit about closing unsaved windows, this isn't what we want to tell the user, we want the user to saved any unsaved data, not just delete it. I also rephrased the first bit to make it orientated to what the user is doing, not speaking about the technical side of how it will work.

I also changed the button text from:
--
No | Yes
--

To:
--
Cancel | Close and Backup Evolution
--

Now there is no confusion over what will happen.



I have changed the restore dialog text from this:
---
This will delete all your current Evolution data and settings and restore them from your backup. Evolution restore can start only when Evolution is not running. Please make sure that you close all your unsaved windows before you proceed. If you want Evolution to restart automatically restart after restore, please enable the toggle button.
---


To this:
---
To restore your data and settings, you must first close Evolution. Please make sure that you save any unsaved data before proceeding. This will delete all your current Evolution data and settings and restore them from your backup.
---

this is the same text as in the backup dialog, but with the added sentence of warning the user of what will happen. Likewise the button text has been changed from:
--
No | Yes
--

To:
--
Cancel | Close and Restore Evolution
--

Can someone apply this patch to trunk?
Comment 3 rugby471 2010-07-28 08:09:52 UTC
Created attachment 166689 [details] [review]
Patch to change the text on both dialogs
Comment 4 rugby471 2010-07-28 08:13:32 UTC
Created attachment 166690 [details] [review]
Patch that works this time

Sorry, made a mistake with the response IDs, here is a patch that works (a bit more useful!)
Comment 5 Matthew Barnes 2010-07-28 20:37:29 UTC
Excellent wording!  Committed your patch for Evolution 2.31.6.

http://git.gnome.org/browse/evolution/commit/?id=a73b369b72ec688c190a32952cd2d672d4eb1bf5
Comment 6 rugby471 2010-07-29 06:49:30 UTC
Thank you very much for the quick response and the kind words :)