Enable the menus API in messages being composed
Categories
(Thunderbird :: Add-Ons: Extensions API, task)
Tracking
(thunderbird_esr78+ fixed)
People
(Reporter: darktrojan, Assigned: darktrojan)
Details
(Whiteboard: [TM:78.6.0])
Attachments
(2 files)
47 bytes,
text/x-phabricator-request
|
wsmwk
:
approval-comm-esr78+
|
Details | Review |
47 bytes,
text/x-phabricator-request
|
wsmwk
:
approval-comm-beta+
wsmwk
:
approval-comm-esr78+
|
Details | Review |
If an extension has the compose permission, it should be able to see things like selected text, image URLs (not that they're much use) when using context menus in messages being composed.
Assignee | ||
Comment 1•4 years ago
|
||
I'm improving this test to prove that the menu in the compose window does the same things as the main context menu, with respect to WebExtensions.
More big changes to the menus code are in the pipeline as well, so locking down the behaviour with a test is good.
Adds tests for the selection, image, and link contexts. Adds tests for other places the main context menu appears.
Assignee | ||
Comment 2•4 years ago
|
||
Depends on D94213
Pushed by geoff@darktrojan.net:
https://hg.mozilla.org/comm-central/rev/3117239e6d5e
Upgrade browser_ext_menus.js some more. r=mkmelin
https://hg.mozilla.org/comm-central/rev/205620c78120
Enable the menus API in messages being composed. r=mkmelin
Assignee | ||
Updated•4 years ago
|
Assignee | ||
Updated•4 years ago
|
Comment 5•4 years ago
|
||
This likely also fixes bug 1656506.
Updated•4 years ago
|
Comment 6•4 years ago
|
||
Comment on attachment 9182790 [details]
Bug 1670832 - Upgrade browser_ext_menus.js some more. r?mkmelin
[Approval Request Comment]
Api fixes. This and the other patch
Comment 7•4 years ago
|
||
Comment on attachment 9182790 [details]
Bug 1670832 - Upgrade browser_ext_menus.js some more. r?mkmelin
[Triage Comment]
Approved for esr78
Comment 8•4 years ago
|
||
Comment on attachment 9182791 [details]
Bug 1670832 - Enable the menus API in messages being composed. r?mkmelin
[Triage Comment]
(to go with the first patch)
Already landed on beta.
Approved for esr78
Comment 9•4 years ago
|
||
bugherder uplift |
Description
•