Closed
Bug 1483088
Opened 6 years ago
Closed 5 years ago
Allow System Addons to Set _insertBeforeActionID
Categories
(WebExtensions :: Frontend, enhancement, P2)
WebExtensions
Frontend
Tracking
(Not tracked)
RESOLVED
INACTIVE
People
(Reporter: bchen, Unassigned)
References
Details
When Screenshots changed from a Photon page action to a WebExtension one, its menu item in the meatball menu moved from the list of built-in actions to the list of alpha-sorted web extensions.
Screenshots' Photon page action had a `_insertBeforeActionID` property that allowed it to be appended to the list of built-in actions (https://dxr.mozilla.org/mozilla-central/source/browser/modules/PageActions.jsm#207).
Perhaps web extensions with the `mozillaAddons` permission should be allowed to set `_insertBeforeActionID`?
Comment 1•6 years ago
|
||
This is within the scope of bug 1370735.
( https://bugzilla.mozilla.org/show_bug.cgi?id=1370735#c3 )
The
Component: Untriaged → Frontend
Comment 2•6 years ago
|
||
Oops, I only wanted to change the component.
Disregard the above comment; I mistook the menu for a context menu but this is about the action button.
Comment 3•6 years ago
|
||
This is being tracked in bug 1483591.
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → DUPLICATE
Comment 4•6 years ago
|
||
(In reply to Mike Conca [:mconca] (Denver, CO, USA UTC-6) from comment #3)
> This is being tracked in bug 1483591.
I'm not sure this is a dupe. Screenshots currently explicitly specifies its insertion position, which means that even if we treat it as if it were a built-in page action, we still need to add an extra manifest property to allow it to specify that position. They should probably be treated as separate bugs.
Reporter | ||
Updated•6 years ago
|
Status: RESOLVED → REOPENED
Resolution: DUPLICATE → ---
Updated•6 years ago
|
Priority: -- → P2
Comment 5•5 years ago
|
||
Is this still needed? If not please close.
Flags: needinfo?(bchen)
Flags: needinfo?(adw)
Reporter | ||
Comment 6•5 years ago
|
||
I haven't been involved with Screenshots for quite some time, so I'm uncertain if it still needs this. Closing.
Status: REOPENED → RESOLVED
Closed: 6 years ago → 5 years ago
Flags: needinfo?(bchen)
Resolution: --- → INACTIVE
Comment 7•5 years ago
|
||
Me either, but this hasn't had any activity in a while.
Flags: needinfo?(adw)
You need to log in
before you can comment on or make changes to this bug.
Description
•