backup settings option uses unhelpful yes/no dialog

Bug #572985 reported by zpletan
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Evolution
Fix Released
Medium
One Hundred Papercuts
Fix Released
Low
Andrew
evolution (Ubuntu)
Fix Released
Low
Unassigned

Bug Description

Binary package hint: evolution

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".

ProblemType: Bug
DistroRelease: Ubuntu 10.04
Package: evolution 2.28.3-0ubuntu9
ProcVersionSignature: Ubuntu 2.6.32-21.32-generic 2.6.32.11+drm33.2
Uname: Linux 2.6.32-21-generic i686
Architecture: i386
Date: Sat May 1 07:06:46 2010
InstallationMedia: Ubuntu 10.04 LTS "Lucid Lynx" - Release i386 (20100429)
ProcEnviron:
 PATH=(custom, user)
 LANG=en_US.utf8
 SHELL=/bin/bash
SourcePackage: evolution

Related branches

Revision history for this message
zpletan (zpletan) wrote :
Revision history for this message
Sebastien Bacher (seb128) wrote :

Thank you for your bug report. The issue is an upstream one and it would be nice if somebody having it could send the bug the to the people writting the software (https://wiki.ubuntu.com/Bugs/Upstream/GNOME)

Changed in evolution (Ubuntu):
importance: Undecided → Low
Revision history for this message
Sam L. (somethinginteres) wrote :

I filed the bug upstream with the GNOME guys.

Changed in evolution:
status: Unknown → New
Revision history for this message
zpletan (zpletan) wrote :

@somethinginteresting,
Thank you! I have no clue how to use GNOME Bugzilla (though that's probably because I've never taken the time to read the linked documents....)

Vish (vish)
Changed in evolution (Ubuntu):
status: New → Triaged
Changed in hundredpapercuts:
importance: Undecided → Low
status: New → Triaged
Vish (vish)
Changed in hundredpapercuts:
milestone: none → maverick-round-2-office
Revision history for this message
Andrew (and471) wrote :

I have sent a patch upstream which does the following:-

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
--

Changed in hundredpapercuts:
assignee: nobody → Andrew (and471)
Changed in evolution (Ubuntu):
assignee: nobody → Andrew (and471)
Andrew (and471)
Changed in hundredpapercuts:
status: Triaged → In Progress
Andrew (and471)
Changed in evolution (Ubuntu):
assignee: Andrew (and471) → nobody
Revision history for this message
Andrew (and471) wrote :
Changed in hundredpapercuts:
status: In Progress → Fix Committed
Revision history for this message
Vish (vish) wrote :

Bugwatch is just being a bit slow!
 But as Andrew mentions , fixed upstream..

Changed in evolution (Ubuntu):
status: Triaged → Fix Committed
Changed in evolution:
status: New → Fix Released
Revision history for this message
我是 Steppenwolf (jado92mx) wrote : Re: [Bug 572985] Re: backup settings option uses unhelpful yes/no dialog

Very good bug correction! Now Evolutions will looks simplier for me and
much, much persons! ;)

Greetings,
JaD!

Changed in evolution:
importance: Unknown → Medium
Revision history for this message
Launchpad Janitor (janitor) wrote :
Download full text (3.3 KiB)

This bug was fixed in the package evolution - 2.32.0-0ubuntu1

---------------
evolution (2.32.0-0ubuntu1) natty; urgency=low

  * Upstream release 2.32.0
    - Empty reply quotation for HTML messages (LP: #630566, #659513)
    - "Encrypt to self" by default on newly created mail (LP: #326979)
    - Backup settings uses unhelpful yes/no dialog (LP: #572985)
    - Can't drag email addresses to Contact List Editor (LP: #282530)
    - Contact List Editor calls wrong EDestination function (LP: #229187)
    - Allow normal, non-vFolder, Trash and Junk folder (LP: #13983, #64762)
      (LP: #135485, #280325, #365270)
    - Allow change of signature hash algorithm (LP: #381290, #381295, #635937)
    - Dialog for mark-all-read always mentions subfolders (LP: #608462)
    - Evolution allows deletion of default views (LP: #498040)
    - Add checks for event->comp_data != NULL (LP: #466415, #546952)
    - Properly free unused message infos periodically (LP: #507972)
    - Calendar compressed weekend print improvement (LP: #88926)
    - Attachment bar causes drawing issues in RTL locales (LP: #545459)
    - [PST] evolution crashed with SIGSEGV (LP: #471852)
    - Swap "Save" and "Save as Draft" accelerators in composer (LP: #424416)
    - Evolution hangs when formatting message - fixes part of it (LP: #175233)
      (LP: #327775)
    - Calendar Day view All Day events print improvements (LP: #88926)
    - Crash on a changed mail filter action removal (LP: #452921)
    - Do not block UI with publish-calendar messages (LP: #594289)
    - Duplicate mnemonic in meeting window (LP: #499418)
    - Hide variable used only with HAVE_LIBNOTIFY (LP: #594289)
  * debian/patches/02_fix_missing_include_for_composer.patch: refreshed
  * debian/patches/03_lpi.patch: refreshed
  * debian/patches/12_remove_not_recommended_for_top_posting.patch: updated.
    The "recommended" text is now a separate label, so removing that object
    entirely.
  * debian/patches/89_express.patch: dropped, applied upstream.
  * debian/patches/90_disable_deprecation_warning.patch: refreshed
  * debian/patches/91_add_u1_email_translations.patch: refreshed
  * debian/patches/91_git_additional_chinese_translations.patch: dropped
  * debian/patches/91_git_fix_e_shell_ref_counting.patch: dropped
  * debian/patches/91_git_fix_untranslatable_list_editor.patch: dropped
  * debian/patches/92_git_fix_proxy_ignore_hosts.patch: dropped
  * debian/patches/92_git_only_hide_signature_on_express.patch: dropped
  * debian/control: update Depends/Build-Depends of all e-d-s parts to 2.32
  * debian/control: libgdata was split out of e-d-s, so updating Build-Depends
    to use the external libgdata library.
  * debian/control: bump libgtkhtml Build-Depends to >= 1:3.31.90
  * debian/*.install: install plugins to /usr/lib/evolution/2.32, not 2.30
  * debian/rules: remove --disable-pilot-conduits, it's not a valid configure
    switch anymore
  * debian/evolution-dev.install: no longer install libeconduit.so
  * debian/control: update Build-Depends with new and updated requirements for
    2.32: adding libpango1.0-dev, libgail-dev, updating libglib2.0-dev,
    libgtk2.0-dev, gnome-icon-theme and libunique-dev
  *...

Read more...

Changed in evolution (Ubuntu):
status: Fix Committed → Fix Released
Vish (vish)
Changed in hundredpapercuts:
milestone: maverick-round-2-office → nt5-office
status: Fix Committed → Fix Released
milestone: nt5-office → none
Revision history for this message
Vish (vish) wrote :

I was reminded that the fix landed in natty.

Changed in hundredpapercuts:
milestone: none → nt5-office
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.