Escape should be equivalent to Cancel in the profile unlock dialog

NEW
Unassigned

Status

()

Toolkit
Startup and Profile System
4 years ago
9 months ago

People

(Reporter: neil@parkwaycc.co.uk, Unassigned)

Tracking

(Depends on: 1 bug, {regression})

Trunk
x86_64
Windows 7
regression
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: needs UX decision)

Attachments

(2 attachments)

(Reporter)

Description

4 years ago
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
(Reporter)

Comment 1

4 years ago
Created attachment 8481365 [details] [diff] [review]
Proposed patch
Assignee: nobody → neil
Status: NEW → ASSIGNED
Attachment #8481365 - Flags: review?(benjamin)

Comment 2

4 years ago
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 3

4 years ago
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)
(Reporter)

Comment 4

4 years ago
(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.
(Reporter)

Comment 5

4 years ago
Created attachment 8483366 [details]
Screen shot
(Reporter)

Comment 6

4 years ago
Whoops, I wasn't thinking straight, and unticked the checkbox to clear the needinfo request...
Flags: needinfo?(neil)

Updated

4 years ago
Attachment #8481365 - Flags: review?(benjamin) → review+
You need to log in before you can comment on or make changes to this bug.