1703889 - Hide disabled page actions in the overflow pageactions menu

Status: RESOLVED FIXED

(Firefox :: Toolbars and Customization, task, P2)

Description

[Marco Bonardo [:mak] (Away Apr 25 - May 5)]

Without Proton, disabled pageactions are shown in the menu, but not pinned of course.
With Proton the menu is gone, so normally disabled page actions are not shown, but in windows smaller than 680px they are shown inside the "overflow" meatball menu. That is inconsistent and Luca in https://phabricator.services.mozilla.com/D111271#inline-620165 suggested we should just hide disabled page action extensions in the overflow menu for Proton urlbar.

That requires to modify some tests (likely something in pageactions-proton/) and add a new test to extensions/test/browser/browser_ext_menus.js to check disabled add-ons are not shown in the menu.

I'm setting [priority:2a] because add-ons team requested this to happen asap.

Comment 1

[Luca Greco [:rpl] [:luca] [:lgreco]]

Attachment: Bug 1703889 - Ensure hidden extension pageAction are not visible in the urlbar overflow menu on Proton. r?mak! r=mixedpuppy

Comment 2

[Luca Greco [:rpl] [:luca] [:lgreco]]

Pushed to try here: https://treeherder.mozilla.org/jobs?repo=try&revision=e3a7eb506e460c005a0f1f4b6a1894ac2fc3e993

I have explicitly tested locally (and fixed if necessary) a bunch of tests that I knew about, and added a new one to explicitly cover the expected behavior, but I wouldn't exclude that there may be some more tests that fails because they still assume that a disabled extension pageAction would be available in the urlbar overflow menu.

Comment 3

[Luca Greco [:rpl] [:luca] [:lgreco]]

(In reply to Luca Greco [:rpl] [:luca] [:lgreco] from comment #2)

Pushed to try here: https://treeherder.mozilla.org/jobs?repo=try&revision=e3a7eb506e460c005a0f1f4b6a1894ac2fc3e993

Well, apparently the only related failure in my push to try is the same one Harry is fixing in Bug 1703838 (which wasn't part yet of the patches I pushed to try), that's good.

And so this is ready for review.

Comment 4

[Luca Greco [:rpl] [:luca] [:lgreco]]

Attachment: Bug 1703889 - Move disabled action fill-opacity rule into non Proton CSS rules. r?mak!

In the Proton UI the disabled extension pageActions are not available in both the urlbar
and the urlbar overflow menu, and non or the builtin actions seems to be visible when
disabled in Proton. And so this particular rule isn't needed in Proton and can be moved
into the group of stylesheet rules restricted to the non-Proton UI.

Depends on D111423

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/f54c679ded81
Ensure hidden extension pageAction are not visible in the urlbar overflow menu on Proton. r=mixedpuppy,mak
https://hg.mozilla.org/integration/autoland/rev/2e13cdc47fb9
Move disabled action fill-opacity rule into non Proton CSS rules. r=mak

Comment 6

[Dorel Luca [:dluca]]

https://hg.mozilla.org/mozilla-central/rev/f54c679ded81
https://hg.mozilla.org/mozilla-central/rev/2e13cdc47fb9