943285 - Add-ons overlaying toolbar buttons directly onto the addon-bar lose their positions because they get evicted every time

Status: RESOLVED FIXED

(Firefox :: Toolbars and Customization, defect)

Description

[:Gijs (he/him)]

(In reply to :Gijs Kruitbosch from bug 854226 comment #22)
> (In reply to klonos from bug 854226 comment #21)
> > I'm seeing various similar issues with addon buttons returning to the same
> > position after browser restart no matter where they were moved (such an
> > example addon is Wappalyzer)
> 
> I had a look, and Wappalyzer directly overlays the add-on bar. I didn't
> think of this case when I wrote the shim that manages stuff that gets added
> to the add-on bar, so it's probably not handled in the best way. I'll
> investigate this further in a separate bug.

Comment 1

[:Gijs (he/him)]

Attachment: fix add-on-bar-bound positioning due to Australis shim,

Here's another thing we should fix about this shim. Simple patch, fortunately. Tested, and works with wappalyzer (you need to toggle the pref to have it show things in its addonbar icon rather than the urlbar icon though).

Comment 2

[Mike Conley (:mconley) (:⚙️)]

Comment on attachment 8338454 [details] [diff] [review]
fix add-on-bar-bound positioning due to Australis shim,

LGTM! Thanks Gijs.

Comment 3

[:Gijs (he/him)]

remote:   https://hg.mozilla.org/integration/fx-team/rev/e1214a4d199b

Comment 4

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/e1214a4d199b