Closed Bug 1026608 Opened 6 years ago Closed 6 years ago

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

Categories

(Thunderbird :: Toolbars and Tabs, defect)

31 Branch
defect
Not set
minor

Tracking

(thunderbird32 fixed, thunderbird_esr3132+ fixed)

RESOLVED FIXED
Thunderbird 33.0
Tracking Status
thunderbird32 --- fixed
thunderbird_esr31 32+ fixed

People

(Reporter: rsx11m.pub, Assigned: aceman)

References

Details

Attachments

(1 file)

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.
Attached 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+
Keywords: checkin-needed
https://hg.mozilla.org/comm-central/rev/eaa66c0f3d6f
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 33.0
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.