YouSendIt password prompt focuses compose windows by default

RESOLVED FIXED in Thunderbird 15.0

Status

Thunderbird
Preferences
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: mconley, Assigned: mconley)

Tracking

Trunk
Thunderbird 15.0
x86
All

Thunderbird Tracking Flags

(thunderbird13 fixed, thunderbird14 fixed)

Details

Attachments

(1 attachment)

(Assignee)

Description

5 years ago
STR:

1)  Open a compose window
2)  Open up the preferences dialog, and choose to create a new YouSendIt account

What happens?

When the password prompt for YouSendIt comes up, it attaches itself to the compose window, which brings the compose window into focus.

What's expected?

YouSendIt's password prompt should attach to the front-most window, in this case.

Comment 1

5 years ago
(In reply to Mike Conley (:mconley) from comment #0)
> STR:
> 
> 1)  Open a compose window
> 2)  Open up the preferences dialog, and choose to create a new YouSendIt
> account
> 
> What happens?
> 
> When the password prompt for YouSendIt comes up, it attaches itself to the
> compose window, which brings the compose window into focus.
Hmm, how the heck does it do that?

the options window is modal, so we'd need to make sure the password prompt works if it's attached to the options window.
(Assignee)

Comment 2

5 years ago
Created attachment 624873 [details] [diff] [review]
Patch v1

Make YouSendIt attach the auth prompt to the most recently used window.
Assignee: nobody → mconley
Attachment #624873 - Flags: review?(dbienvenu)

Comment 3

5 years ago
Comment on attachment 624873 [details] [diff] [review]
Patch v1

this is likely to be right 99% of the time. I can imagine cases where it's not (start an attach, switch windows really quickly, have the prompt come up for the switched to window) but the code as it was is always wrong for the options ui.
Attachment #624873 - Flags: review?(dbienvenu) → review+
(Assignee)

Comment 4

5 years ago
Comment on attachment 624873 [details] [diff] [review]
Patch v1

This is super low-risk. Probably worth landing in TB 13/14.
Attachment #624873 - Flags: approval-comm-beta?
Attachment #624873 - Flags: approval-comm-aurora?

Updated

5 years ago
Attachment #624873 - Flags: approval-comm-beta?
Attachment #624873 - Flags: approval-comm-beta+
Attachment #624873 - Flags: approval-comm-aurora?
Attachment #624873 - Flags: approval-comm-aurora+
(Assignee)

Comment 5

5 years ago
comm-central: https://hg.mozilla.org/comm-central/rev/250046d3b130
comm-aurora: https://hg.mozilla.org/releases/comm-aurora/rev/3931a8792db6
comm-beta: https://hg.mozilla.org/releases/comm-beta/rev/fb90c6dc325a
Status: NEW → RESOLVED
Last Resolved: 5 years ago
status-thunderbird13: --- → fixed
status-thunderbird14: --- → fixed
OS: Mac OS X → All
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 15.0
You need to log in before you can comment on or make changes to this bug.