Closed Bug 1670832 Opened 4 years ago Closed 4 years ago

Enable the menus API in messages being composed

Categories

(Thunderbird :: Add-Ons: Extensions API, task)

Tracking

(thunderbird_esr78+ fixed)

RESOLVED FIXED
84 Branch
Tracking Status
thunderbird_esr78 + fixed

People

(Reporter: darktrojan, Assigned: darktrojan)

Details

(Whiteboard: [TM:78.6.0])

Attachments

(2 files)

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.

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.

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

Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → 84 Branch
Pushed by geoff@darktrojan.net:
https://hg.mozilla.org/comm-central/rev/4fd365d76ca4
follow-up - fix formatting error. rs=linting DONTBUILD
Whiteboard: [TM:78.6.0]

This likely also fixes bug 1656506.

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

Attachment #9182790 - Flags: approval-comm-esr78?

Comment on attachment 9182790 [details]
Bug 1670832 - Upgrade browser_ext_menus.js some more. r?mkmelin

[Triage Comment]
Approved for esr78

Attachment #9182790 - Flags: approval-comm-esr78? → approval-comm-esr78+

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

Attachment #9182791 - Flags: approval-comm-esr78+
Attachment #9182791 - Flags: approval-comm-beta+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: