[commands] Support dynamic commands
Categories
(WebExtensions :: General, enhancement)
Tracking
(Not tracked)
People
(Reporter: ryan.hendrickson, Unassigned, Mentored)
References
Details
(Whiteboard: [design-decision-approved] triaged [commands])
Attachments
(1 file)
(deleted),
patch
|
Details | Diff | Splinter Review |
Comment 1•8 years ago
|
||
Comment 2•7 years ago
|
||
Updated•7 years ago
|
Comment 3•7 years ago
|
||
Updated•7 years ago
|
Updated•7 years ago
|
Comment 4•7 years ago
|
||
Comment 5•7 years ago
|
||
Comment 6•7 years ago
|
||
Comment 7•7 years ago
|
||
Comment 8•7 years ago
|
||
Comment 9•7 years ago
|
||
Comment 10•7 years ago
|
||
Comment 11•7 years ago
|
||
Comment 12•7 years ago
|
||
Comment 13•7 years ago
|
||
Comment 14•7 years ago
|
||
Comment 15•7 years ago
|
||
Comment 16•7 years ago
|
||
Comment 17•7 years ago
|
||
Comment 18•7 years ago
|
||
Comment 19•7 years ago
|
||
Updated•6 years ago
|
Comment 20•6 years ago
|
||
Updated•4 years ago
|
Reporter | ||
Comment 22•4 years ago
|
||
Bug 1421811 is about changing the shortcuts of existing commands. You can approximate dynamic commands with that by shipping a static list of unused commands, but then you're limited to that number of extra commands. I opened this bug explicitly to ask for something that wouldn't have such a limitation; I don't think this should be considered a dupe of bug 1421811.
Comment 23•4 years ago
|
||
Thanks for the clarification - I now see that you're asking for a dynamic number of commands rather than a dynamic shortcut.
On the one hand I see that it may be useful to not be constrained by the number of shortcuts in manifest.json.
On the other hand, an effective way to counter abuse (and help with auditing) is to require extensions to declare shortcuts in manifest.json.
I'll clear the priority to reassess this bug at the next bug triage.
Comment 24•4 years ago
|
||
Declarative shortcuts are much more manageable, and we don't wanna do this without a good use case not covered by manifest commands.
Comment 25•4 years ago
|
||
That having said, if you have specific issues caused by the current implementation (e.g. a desire to hide unused/empty shortcuts from about:addons
), then we will consider that, as a new feature request.
Description
•