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)

enhancement
Not set
normal

Tracking

(thunderbird74+ fixed)

RESOLVED FIXED
Thunderbird 75.0
Tracking Status
thunderbird74 + fixed

People

(Reporter: darktrojan, Assigned: darktrojan)

Details

Attachments

(1 file)

There's two issues here:

  1. browser.composeAction.openPopup doesn't work at all.
  2. onBeforeSend isn't considered a user action so openPopup can't be called from it.

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)
Attachment #9128638 - Flags: approval-comm-beta?
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+
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

Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: