Closed
Bug 1617742
Opened 4 years ago
Closed 4 years ago
It should be possible to open a composeAction popup from onBeforeSend
Categories
(Thunderbird :: Add-Ons: Extensions API, enhancement)
Thunderbird
Add-Ons: Extensions API
Tracking
(thunderbird74+ fixed)
RESOLVED
FIXED
Thunderbird 75.0
People
(Reporter: darktrojan, Assigned: darktrojan)
Details
Attachments
(1 file)
1.97 KB,
patch
|
mkmelin
:
review+
mkmelin
:
approval-comm-beta+
|
Details | Diff | Splinter Review |
There's two issues here:
browser.composeAction.openPopup
doesn't work at all.onBeforeSend
isn't considered a user action soopenPopup
can't be called from it.
Assignee | ||
Comment 1•4 years ago
|
||
Finding the right window to open the popup in is a bit ugly (XPCOM window handling functions are badly lacking some useful things IMO) but any window other than the most recent is unlikely to be the right window.
Attachment #9128638 -
Flags: review?(mkmelin+mozilla)
Assignee | ||
Updated•4 years ago
|
Attachment #9128638 -
Flags: approval-comm-beta?
Comment 2•4 years ago
|
||
Comment on attachment 9128638 [details] [diff] [review] 1617742-onbeforesend-openpopup-1.diff Review of attachment 9128638 [details] [diff] [review]: ----------------------------------------------------------------- LGTM, r=mkmelin
Attachment #9128638 -
Flags: review?(mkmelin+mozilla)
Attachment #9128638 -
Flags: review+
Attachment #9128638 -
Flags: approval-comm-beta?
Attachment #9128638 -
Flags: approval-comm-beta+
Updated•4 years ago
|
Keywords: checkin-needed-tb
Target Milestone: --- → Thunderbird 75.0
Pushed by mkmelin@iki.fi:
https://hg.mozilla.org/comm-central/rev/f34fd430a920
Enable opening of composeAction popups by onBeforeSend listener. r=mkmelin
Updated•4 years ago
|
status-thunderbird74:
--- → affected
tracking-thunderbird74:
--- → +
Comment 4•4 years ago
|
||
bugherder uplift |
Thunderbird 74.0b2:
https://hg.mozilla.org/releases/comm-beta/rev/2aeba9647436
Updated•4 years ago
|
You need to log in
before you can comment on or make changes to this bug.
Description
•