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 726290 - Composer doesn't autosave draft if only envelope altered
Composer doesn't autosave draft if only envelope altered
Status: RESOLVED FIXED
Product: geary
Classification: Other
Component: composer
master
Other Linux
: High normal
: 0.12.0
Assigned To: Geary Maintainers
Geary Maintainers
Depends on: geary-wk2
Blocks:
 
 
Reported: 2014-03-13 22:15 UTC by Jim Nelson
Modified: 2017-01-31 23:43 UTC
See Also:
GNOME target: ---
GNOME version: ---


Description Jim Nelson 2014-03-13 22:15:15 UTC 
This looks similar to bug #724911:

* Create a new draft and save it
* Go to Drafts folder and select new message
* Press "Edit Draft"
* Edit envelope (From, To, Cc, Bcc, Subject)

At this point, I expect the composer to autosave the changes to the draft, but it doesn't.  Additionally, if I close the composer, it does not offer Keep/Discard and the changes are lost.
Comment 1 Jim Nelson 2014-03-13 23:32:50 UTC 
As discussed when Charles fixed bug #724911, that was merely to ensure that the draft is only created when the body is filled in, that is, not if only the envelope was touched.  This didn't seem too bad for creating a new draft.

For editing a draft, however, it seems a little more troublesome.  It's conceivable that a user would save a draft, edit it to add someone to the cc: list or change the subject, then close the composer -- only to see that those changes were not saved, and worse, they cannot even manually save them (as there's no Save button).

So, this ticket is changed to save drafts when the envelope is altered in any way.  That's too much for the 0.6 cycle, so moving to the next release.
Comment 2 Michael Gratton 2017-01-16 04:56:14 UTC 
This has been fixed on the WK2 branch and will land in master when that is merged.
Comment 3 Michael Gratton 2017-01-31 23:43:52 UTC 
Fixed on master with commit 29b339c.