GNOME Bugzilla – Bug 546551
Dialog for mark-all-read always mentions subfolders
Last modified: 2011-02-02 14:23:30 UTC
0) The dialog for deleting folders ("ask-delete-folder" in mail/mail.error.xml) always mentions subfolders: <error id="ask-delete-folder" type="question" default="GTK_RESPONSE_CANCEL"> <_title>Delete "{0}"?</_title> <_primary>Really delete folder "{0}" and all of its subfolders?</_primary> <_secondary xml:space="preserve">If you delete the folder, all of its contents and its subfolders contents will be deleted permanently.</_secondary> <button stock="gtk-cancel" response="GTK_RESPONSE_CANCEL"/> <button stock="gtk-delete" response="GTK_RESPONSE_OK"/> </error> 1) It should be checked whether there actually are subfolders in a folder to be deleted. (Probably like the mark-all-read plugin does in mark-all-read/mark-all-read.c:mar_got_folder().) 2) The dialog should (I suppose) be split in two different dialogs (one for each situation). 3) Sorry, little time to work on that patch now.
Confirming.
Similar too bug #518919 (which might be more general than just IMAP).
Similar issue: the ask-mark-all-read dialog (invoked by Folder > "Mark All Messages as Read"): <error id="ask-mark-all-read" type="question" default="GTK_RESPONSE_NO"> <_title>Mark all messages as read</_title> <_primary>Do you want to mark all messages as read?</_primary> <_secondary xml:space="preserve">This will mark all messages as read in the selected folder and its subfolders.</_secondary> <button stock="gtk-no" response="GTK_RESPONSE_NO"/> <button stock="gtk-yes" response="GTK_RESPONSE_YES"/> </error>
Created attachment 151459 [details] [review] Proposed Patch
Patch looks okay to me. Would be nice to handle comment #3 too (not necessarily detect whether there's unread mail in subfolders, but just whether there -are- subfolders).
Comment on attachment 151459 [details] [review] Proposed Patch Committed to master. http://git.gnome.org/browse/evolution/commit/?id=8785fd3315996dffceec133f94096577a5547cbc
Changing summary accordingly.
*** Bug 617624 has been marked as a duplicate of this bug. ***
Downstream bug report about the same: https://bugzilla.redhat.com/show_bug.cgi?id=593874
Created attachment 161648 [details] [review] evo patch for evolution; mark-all-read part.
Created commit a5d79e9 in evo master (2.31.2+)
*** Bug 622363 has been marked as a duplicate of this bug. ***
*** Bug 623283 has been marked as a duplicate of this bug. ***
*** Bug 623282 has been marked as a duplicate of this bug. ***
*** Bug 624862 has been marked as a duplicate of this bug. ***
*** Bug 625075 has been marked as a duplicate of this bug. ***
*** Bug 625035 has been marked as a duplicate of this bug. ***
*** Bug 630634 has been marked as a duplicate of this bug. ***
*** Bug 632385 has been marked as a duplicate of this bug. ***
*** Bug 641234 has been marked as a duplicate of this bug. ***