The “Remove Extension” context menu item is not available for pageAction
Categories
(WebExtensions :: Frontend, defect, P3)
Tracking
(firefox62 disabled, firefox63 disabled, firefox64 wontfix, firefox77 fixed)
People
(Reporter: cbadescu, Assigned: willdurand)
References
Details
Attachments
(2 files)
[Affected versions]: - Firefox 64.0a1(20180906232139) - Firefox 63.b04(20180906162647) - Firefox 62.0(20180830143136) [Affected platforms]: - Win 7 64-bit - Mac OS X 10.13.3 [Steps to reproduce]: 1.Install some extensions that use pageAction. 2.Right-click on the pageAction icon. 3.Observe there is no “Remove Extension” option. [Expected results]: - The “Remove Extension” context menu item to be present in the context menu. [Actual results]: - The “Remove Extension” context menu item is only available for the browserAction icon. Please see the attached video.
Updated•6 years ago
|
Updated•4 years ago
|
Assignee | ||
Updated•4 years ago
|
Comment 1•4 years ago
|
||
Hey Will, thanks for taking this issue, here is some pointers that may be useful to find some of the relevant pieces:
-
the pageAction context menu item actions are defined in browser/base/content/browser-pageActions.js, e.g. here is the action for "Manage Extension"
-
the pageAction localized strings are defined in the browser.ftl fluent file
-
the pageAction HTML DOM elements are defined in browser.xhtml
-
the browserAction's "Remote Extension" context menu item is currently defined here
I think that the pageAction would likely have to do most of what ToolbarContextMenu.removeExtensionForContextAction
is doing for the browserAction context menu item, and so it would be great if we could share at least part of the code to handle the extension context menu item shared by both pageAction and browserAction (Mark Striemer may help you to evaluate that or find someone from Firefox Frontend that would be good to consult on the topic).
Assignee | ||
Comment 2•4 years ago
|
||
Pushed by cbrindusan@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/526c65b7b516 Add "Remove Extension" context menu item for pageAction. r=robwu,mstriemer,fluent-reviewers,Gijs
Comment 4•4 years ago
|
||
bugherder |
Updated•4 years ago
|
Description
•