Closed
Bug 588478
Opened 14 years ago
Closed 14 years ago
Escape should not open tab from fan view of stacked tabs
Categories
(Firefox Graveyard :: Panorama, defect)
Firefox Graveyard
Panorama
Tracking
(Not tracked)
VERIFIED
FIXED
Firefox 4.0b5
People
(Reporter: tracy, Assigned: raymondlee)
Details
(Whiteboard: [4b4][in-litmus-bug-week])
Attachments
(1 file, 2 obsolete files)
1.28 KB,
patch
|
Details | Diff | Splinter Review |
Seen on Firefox4 beta4 (build 2) all platforms STR: 1) in Tab View, reduce a tab group size until the tabs go into stacked mode 2) open the fan view of the stack by clicking the icon 3) press the escape key Tested results: That tab group is opened in the browser. (this is "enter/return" behavior) Expected: The fan view dismisses back to stacked, we remain in Tab View
Assignee | ||
Comment 1•14 years ago
|
||
Attachment #467351 -
Flags: review?(dolske)
Attachment #467351 -
Flags: feedback?(ian)
Assignee | ||
Updated•14 years ago
|
Assignee: nobody → raymond
Comment 2•14 years ago
|
||
Comment on attachment 467351 [details] [diff] [review] v1 Looks good to me; let's do it. Aza, chime in if this UI tweak is not to your liking.
Attachment #467351 -
Flags: feedback?(ian) → feedback+
Updated•14 years ago
|
Attachment #467351 -
Flags: review?(dolske)
Attachment #467351 -
Flags: review+
Attachment #467351 -
Flags: approval2.0+
Assignee | ||
Comment 3•14 years ago
|
||
Attachment #467351 -
Attachment is obsolete: true
Assignee | ||
Updated•14 years ago
|
Keywords: checkin-needed
Comment 4•14 years ago
|
||
Raymond, you attached the wrong patch here
Assignee | ||
Comment 5•14 years ago
|
||
Attachment #468900 -
Attachment is obsolete: true
Comment 6•14 years ago
|
||
http://hg.mozilla.org/mozilla-central/rev/88adb1e400a8
Status: NEW → RESOLVED
Closed: 14 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 4.0b5
Reporter | ||
Comment 7•14 years ago
|
||
verified with Mozilla/5.0 (Macintosh; Intel Mac OS X 10.6; rv:2.0b5pre) Gecko/20100825 Minefield/4.0b5pre
Status: RESOLVED → VERIFIED
Reporter | ||
Comment 8•14 years ago
|
||
Flagging it to pick up during Bug Week (9-13 to 9-17). Just an addition to the stacking test case.
Flags: in-litmus?
Comment 9•14 years ago
|
||
Added in Litmus: https://litmus.mozilla.org/show_test.cgi?searchType=by_id&id=12848
Flags: in-litmus? → in-litmus+
Whiteboard: [4b4] → [4b4][in-litmus-bug-week]
Updated•8 years ago
|
Product: Firefox → Firefox Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•