Closed Bug 1483088 Opened 6 years ago Closed 5 years ago

Allow System Addons to Set _insertBeforeActionID

Categories

(WebExtensions :: Frontend, enhancement, P2)

enhancement

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`?
Blocks: 1466575
Blocks: 1422437
Component: Untriaged → Frontend
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.
This is being tracked in bug 1483591.
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → DUPLICATE
(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.
Status: RESOLVED → REOPENED
Resolution: DUPLICATE → ---
Blocks: 1483591
Priority: -- → P2

Is this still needed? If not please close.

Flags: needinfo?(bchen)
Flags: needinfo?(adw)

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 ago5 years ago
Flags: needinfo?(bchen)
Resolution: --- → INACTIVE

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.