Closed
Bug 482801
Opened 17 years ago
Closed 17 years ago
Middle-clicking the selected tab should not focus the tab bar
Categories
(Firefox :: Tabbed Browser, defect)
Firefox
Tabbed Browser
Tracking
()
VERIFIED
FIXED
Firefox 3.6a1
People
(Reporter: j.moz, Assigned: dao)
References
Details
(Keywords: verified1.9.1)
Attachments
(1 file)
|
789 bytes,
patch
|
Gavin
:
review+
beltzner
:
approval1.9.1+
|
Details | Diff | Splinter Review |
Follow-up to bug 462289:
Middle-clicking the selected tab still focuses the tab bar on mousedown, before closing the tab on mouseup. The tab bar should not be focused.
Steps to reproduce:
1) Click (and don't release) the current tab on the tab bar with the *middle* mouse button, don't release the button yet.
What happens: the tab bar is focused
What should've happened: nothing yet
(you can release the middle button now, and the tab closes)
| Assignee | ||
Updated•17 years ago
|
Updated•17 years ago
|
Severity: normal → trivial
Better steps to reproduce:
1) Middle-click and hold (start "middle-dragging") the current tab
2) Move the mouse cursor down until it's over the page content
3) Release the middle mouse button
What happens: the tab bar is focused
What should've happened: nothing
| Assignee | ||
Updated•17 years ago
|
Assignee: nobody → dao
| Assignee | ||
Comment 2•17 years ago
|
||
It's not clear to me why style updates need to be flushed there, I've just copied this from the other handler.
I'd like to get this into 3.5 for bug 490465.
Attachment #375190 -
Flags: review?(gavin.sharp)
Updated•17 years ago
|
Attachment #375190 -
Flags: review?(gavin.sharp) → review+
| Assignee | ||
Updated•17 years ago
|
Keywords: checkin-needed
| Assignee | ||
Comment 3•17 years ago
|
||
Status: NEW → RESOLVED
Closed: 17 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 3.6a1
| Assignee | ||
Updated•17 years ago
|
Attachment #375190 -
Flags: approval1.9.1?
Comment 4•17 years ago
|
||
Comment on attachment 375190 [details] [diff] [review]
patch
a191=beltzner. This also needs a test.
Attachment #375190 -
Flags: approval1.9.1? → approval1.9.1+
| Assignee | ||
Updated•17 years ago
|
Keywords: checkin-needed
| Assignee | ||
Comment 5•17 years ago
|
||
Flags: in-testsuite+
| Assignee | ||
Comment 6•17 years ago
|
||
Keywords: checkin-needed → fixed1.9.1
Comment 7•17 years ago
|
||
verified FIXED on builds:
Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.1pre) Gecko/20090604 Shiretoko/3.5pre (.NET CLR 3.5.30729) ID:20090604045218
and
Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.2a1pre) Gecko/20090604 Minefield/3.6a1pre (.NET CLR 3.5.30729) ID:20090604042555
Status: RESOLVED → VERIFIED
Keywords: fixed1.9.1 → verified1.9.1
You need to log in
before you can comment on or make changes to this bug.
Description
•