Delete button on main toolbar doesn't always change to "Undelete" when toggled from keyboard or header pane

RESOLVED FIXED in Thunderbird 33.0

Status

defect
--
minor
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: rsx11m.pub, Assigned: aceman)

Tracking

31 Branch
Thunderbird 33.0

Thunderbird Tracking Flags

(thunderbird32 fixed, thunderbird_esr3132+ fixed)

Details

Attachments

(1 attachment)

(Reporter)

Description

5 years ago
That's a fringe use case but a bug anyway:

 - Configuration:
   - Set up Thunderbird for an IMAP account
   - Select "Mark as deleted" in Server Settings
   - Put the "Delete" button on the main toolbar

 - Go to the last message of your Inbox or any other IMAP folder
   (that's to prevent it to moving to the next message after delete)

 - Click "Delete" in the header-pane toolbar, OR
 - hit the "Delete" key of your keyboard

Expected result:

 - Main toolbar button changes to "Undelete" per bug 234665

Actual result:

 - Main toolbar remains with "Delete" label

 - On top, toggle "Delete" status again (not from the main toolbar),
   observe that the toolbar button changes but to the /opposite/ state

The label is correct when /first/ entering a marked-as-deleted message (see bug 801264 though, which may be related).

BTW: The header-pane "Delete" button stays as such and doesn't change to "Undelete" with the message stricken out, that's on purpose?
 
Observed while testing 31.0b1 on Linux, but also seen with 24.6.0 release and on Windows, so this has been in there for a while.

SeaMonkey 2.26.1 does not show the issue, thus Thunderbird-specific.
(Assignee)

Comment 1

5 years ago
Posted patch patchSplinter Review
This works for me. It looks like not all of the changes are really needed to fix this bug, but I added it to all delete paths where it was missing to have some consistency.

This does not fix the message header button, but that one does not have the infrastructure for Delete/Undelete as the toolbar button has.
Assignee: nobody → acelists
Status: NEW → ASSIGNED
Attachment #8443902 - Flags: review?(mkmelin+mozilla)
Attachment #8443902 - Flags: review?(mkmelin+mozilla) → review+
(Assignee)

Updated

5 years ago
Keywords: checkin-needed
https://hg.mozilla.org/comm-central/rev/eaa66c0f3d6f
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 33.0
(Reporter)

Comment 3

5 years ago
Is this wanted for the new 31.x branch?
Seems to be easy enough as a patch...
Please can someone request relevant approvals (beta & esr?) with appropriate risk statements.
Comment on attachment 8443902 [details] [diff] [review]
patch

Ok, I normally like a bit of analysis, but this seems simple enough we can just move forward on. Lets get this onto aurora and beta for some additional testing.
Attachment #8443902 - Flags: approval-comm-esr31?
Attachment #8443902 - Flags: approval-comm-beta+
Attachment #8443902 - Flags: approval-comm-aurora+
Comment on attachment 8443902 [details] [diff] [review]
patch

This already landed in 33.
Attachment #8443902 - Flags: approval-comm-aurora+
Attachment #8443902 - Flags: approval-comm-esr31? → approval-comm-esr31+
You need to log in before you can comment on or make changes to this bug.