The default bug view has changed. See this FAQ.

Quick Filter Toggle is gone (or doesn't move from tabbar toolbar) on upgrade to TB 11 with Enigmail and/or Lightning installed

RESOLVED FIXED

Status

Thunderbird
Toolbars and Tabs
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: mconley, Unassigned)

Tracking

11 Branch
x86
All
Dependency tree / graph

Thunderbird Tracking Flags

(thunderbird11+ fixed, thunderbird12+ fixed)

Details

(Whiteboard: [fixed by changes in extensions])

Attachments

(1 attachment)

Bug 717264, which moves the Quick Filter Bar toggle from the tabbar-toolbar to mail-bar3, landed earlier this week, and we're getting sporadic reports that the toggle isn't moving OR is disappearing in some cases.

Chris might correct me here, but I think the steps to reproduce are as follows:

1)  In a recent TB 11, install Enigmail or Lightning in a new profile.
2)  Set mail.ui-rdf.version to 2, and shut down Thunderbird
3)  Replace the localstore.rdf for the new profile with the one attached to this bug
4)  Start up TB

What happens?

According to Chris Coulson, with Lightning installed the toggle stays where it is.  With Enigmail installed, the toggle disappears from the toolbars.

What's expected?

The QFB toggle should be in the mail toolbar (mail-bar3).

Reports for this have cropped up for both OSX and Linux, so I'll assume it affects all platforms.
Created attachment 600141 [details]
Localstore.rdf for testing
Blocks: 717264
Ludo:

Do you know any Enigmail hackers we should CC on this?

-Mike
Depends on: 730078

Updated

5 years ago
tracking-thunderbird11: --- → ?
tracking-thunderbird12: --- → ?
tracking-thunderbird13: --- → ?
tracking-thunderbird11: ? → +
tracking-thunderbird12: ? → +
tracking-thunderbird13: ? → +

Comment 3

5 years ago
This doesn't look like a bug with the migration code. As Mike pointed out on IRC, there are no settings for mail-bar3 or tabbar-toolbar in my localstore.rdf, but DOM inspector seems to show that the QFB is still in the defaultset for the tabbar-toolbar after the upgrade

Comment 4

5 years ago
This is why:

http://mxr.mozilla.org/comm-central/source/calendar/lightning/content/messenger-overlay-sidebar.xul#123
The lightning part is covered in bug 730078, but I'm tracking this bug until we get that fixed so that we know we're a bit more covered for the release.

Comment 6

5 years ago
Thanks. I've just attached a patch which doesn't hardcode the defaultset at all in lightning

Comment 7

5 years ago
I fixed this for Enigmail. But I have two requests for the future:

(1) Given the 6 weeks release cycle I would really appreciate if there were no such late changes on comm-beta. I need 3 weeks to obtain review approval on AMO(!), therefore I am already in the process of creating a new release; now I'll have to restart, and the deadline is almost impossible to reach.

(2) In the past, changes to mail-bar would have caused the ID to change (e.g. mail-bar2 to mail-bar3). This would allow add-ons to behave correctly for several versions of Thunderbird.
Patrick:


(In reply to Patrick Brunschwig from comment #7)
> I fixed this for Enigmail. But I have two requests for the future:
> 

Thank you!

> Given the 6 weeks release cycle I would really appreciate if there were no such 
> late changes on comm-beta.

Absolutely, and I apologize (and agree!) whole-heartedly.

I should have pushed back on this change more.  Next time, I'll push back harder - because you're right, it's really not fair for add-on developers to change things like this in the 11th hour.

> (2) In the past, changes to mail-bar would have caused the ID to change
> (e.g. mail-bar2 to mail-bar3). This would allow add-ons to behave correctly
> for several versions of Thunderbird.

I believe changing the ID would cause your overlays, and any code that queries for the mail bar at runtime to fail.

It would also completely wipe-out persisted user-customizations for that toolbar.

We've adapted Firefox's approach to this, which is to run a migration function on startup, that - if necessary - makes the appropriate changes to the toolbars, instead of using the nuclear option (changing the toolbar ID).

Thanks for your work,

-Mike
The lightning part is now pushed, I'll be spinning a beta build tomorrow.
So as far as we know, this is now fixed because the extensions have been fixed. Hence closing the bug down.
Status: NEW → RESOLVED
Last Resolved: 5 years ago
status-thunderbird11: --- → fixed
status-thunderbird12: --- → fixed
tracking-thunderbird13: + → ---
Resolution: --- → FIXED
Whiteboard: [fixed by changes in extensions]
You need to log in before you can comment on or make changes to this bug.