Closed
Bug 482801
Opened 16 years ago
Closed 16 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•16 years ago
|
Updated•16 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•16 years ago
|
Assignee: nobody → dao
Assignee | ||
Comment 2•16 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•16 years ago
|
Attachment #375190 -
Flags: review?(gavin.sharp) → review+
Assignee | ||
Updated•16 years ago
|
Keywords: checkin-needed
Assignee | ||
Comment 3•16 years ago
|
||
Status: NEW → RESOLVED
Closed: 16 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 3.6a1
Assignee | ||
Updated•16 years ago
|
Attachment #375190 -
Flags: approval1.9.1?
Comment 4•16 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•16 years ago
|
Keywords: checkin-needed
Assignee | ||
Comment 5•16 years ago
|
||
Flags: in-testsuite+
Assignee | ||
Comment 6•16 years ago
|
||
Keywords: checkin-needed → fixed1.9.1
Comment 7•16 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
•