Open Bug 1060424 Opened 7 years ago Updated 4 years ago

Escape should be equivalent to Cancel in the profile unlock dialog

Categories

(Toolkit :: Startup and Profile System, defect)

x86_64
Windows 7
defect
Not set
normal

Tracking

()

People

(Reporter: neil, Unassigned)

References

(Depends on 1 open bug)

Details

(Keywords: regression, Whiteboard: needs UX decision)

Attachments

(2 files)

Steps to reproduce problem:
1. Accidentally launch firefox -no-remote when it is already running
2. Notice that the profile unlocker now offers to kill the existing process
3. Press Escape assuming it will have the same effect as Cancel

Actual result: Existing Firefox process is killed

Expected result: Profile unlock dialog is cancelled
Attached patch Proposed patchSplinter Review
Assignee: nobody → neil
Status: NEW → ASSIGNED
Attachment #8481365 - Flags: review?(benjamin)
I agree this is probably correct, but I'd like to get the UX decision as part of bug 1057052.
Assignee: neil → nobody
Status: ASSIGNED → NEW
Depends on: 1057052
Keywords: regression
Whiteboard: needs UX decision
Comment on attachment 8481365 [details] [diff] [review]
Proposed patch

Can you attach a screenshot of what this looks like on windows now? If I'm reading this right, you're changing the button ordering, and removing BUTTON_POS_1_DEFAULT, and I'm not sure how this affects the appearance and the default button styling (if there is a default button now that is activated with enter?)
Flags: needinfo?(neil)
(In reply to Benjamin Smedberg from comment #3)
> Can you attach a screenshot of what this looks like on windows now?
I'll get you one tomorrow, I only have Windows 2003 right now.

> If I'm reading this right, you're changing the button ordering, and
> removing BUTTON_POS_1_DEFAULT, and I'm not sure how this affects the
> appearance and the default button styling (if there is a default button
> now that is activated with enter?)
The default default button is button 0. This is now the button labelled "Close %S". Button 1 is now labelled "Cancel". Pressing Escape is always equivalent to button 1.
Attached image Screen shot
Whoops, I wasn't thinking straight, and unticked the checkbox to clear the needinfo request...
Flags: needinfo?(neil)
Attachment #8481365 - Flags: review?(benjamin) → review+
You need to log in before you can comment on or make changes to this bug.