toolkit.tabbox.switchByScrolling:true doesn't switch tabs when scrolling over empty space in tab bar
Categories
(Toolkit :: XUL Widgets, defect, P3)
Tracking
()
Tracking | Status | |
---|---|---|
firefox-esr68 | --- | unaffected |
firefox71 | --- | disabled |
firefox72 | --- | disabled |
firefox73 | --- | fixed |
People
(Reporter: yoasif, Assigned: emilio)
References
(Regression)
Details
(Keywords: regression)
Attachments
(2 files)
As reported on https://www.reddit.com/r/firefox/comments/e5lojt/tab_wheel_scroll_changed_in_firefox_71/
Steps to reproduce:
- Set toolkit.tabbox.switchByScrolling to true
- Restart Firefox (not sure this is required)
- Open three tabs in a maximized window (doesn't need to be maximized, but the window has to be wider than the width of the tabs)
- Hover over empty space to the right of the tabs, scroll up and down
What happens:
Nothing.
Expected result:
The same result as hovering over tabs or prior to bug 1514926
37:01.43 INFO: No more inbound revisions, bisection finished.
37:01.43 INFO: Last good revision: 94eaa02e0ea1725512893cd5139b1d200fcf4fdd
37:01.43 INFO: First bad revision: 5e221240fc1d537da612973800bbe24efa50736b
37:01.43 INFO: Pushlog:
https://hg.mozilla.org/integration/autoland/pushloghtml?fromchange=94eaa02e0ea1725512893cd5139b1d200fcf4fdd&tochange=5e221240fc1d537da612973800bbe24efa50736b
Reporter | ||
Updated•5 years ago
|
Reporter | ||
Updated•5 years ago
|
Comment 1•5 years ago
|
||
This event (https://searchfox.org/mozilla-central/rev/efdf9bb55789ea782ae3a431bda6be74a87b041e/toolkit/content/widgets/tabbox.js#463) isn't retargeting outside of the tabbrowser-tabs arrowscrollbox shadow DOM (similar to these events: https://searchfox.org/mozilla-central/rev/efdf9bb55789ea782ae3a431bda6be74a87b041e/browser/base/content/tabbrowser-tabs.js#961).
We could either listen there in the tabbrowser-tabs arrowScrollbox and dispatch the event from the Shadow DOM, or make a new function and call it from there. Though I suspect this would continue to happen if/when other components start to use SD (i.e. <tab> contents). So it seems better to switch to the standard "wheel" event instead: https://developer.mozilla.org/en-US/docs/Web/API/Element/wheel_event
Comment 3•5 years ago
|
||
Hm, switching to "wheel" has different behavior, it fires much more often so it rotates too quickly through the tabs.
Emilio, do you have any ideas here, or know who would? Basically we are currently relying on the non standard DOMMouseScroll to implement this (off by default) feature in tabs where scrolling changes the selected tab. The problem is that this event gets swallowed by the tabbrowser-tabs child arrowscrollbox's Shadow DOM (when you scroll to the right of any <tab> children), so it never fires on it.
The event is bound in the custom element at https://searchfox.org/mozilla-central/rev/efdf9bb55789ea782ae3a431bda6be74a87b041e/toolkit/content/widgets/tabbox.js#463. Switching this to "wheel" instead works but has weird behavior. Is there a way to capture the DOMMouseScroll from the parent of the shadow host?
Assignee | ||
Comment 4•5 years ago
|
||
Makes sense to be consistent with wheel.
Updated•5 years ago
|
Assignee | ||
Comment 5•5 years ago
|
||
Does the patch above fix it? (I can't build it atm, on a pretty shitty laptop).
In general, it'd be great if we used wheel and throttled it (which should be easy enough). But these events should probably be consistent with wheel.
I'll send for review if they fix the issue, otherwise I'll move to a different bug.
Comment 6•5 years ago
|
||
Updating affected flags.
Updated•5 years ago
|
Comment 8•5 years ago
|
||
P3 and needs a manual preference flip to show the bug, wontfix 71.
Tell please, in which version of Firefox will this problem be fixed? I really need the tab scrolling function for any number of them.
Assignee | ||
Comment 10•5 years ago
|
||
(In reply to veron from comment #9)
Tell please, in which version of Firefox will this problem be fixed? I really need the tab scrolling function for any number of them.
The patch needs to be reviewed, then landed. That should happen pretty soon and then it'd land in Nightly. Not sure if we'd uplift the patch for 72, as it's a web-exposed change and thus there's some risk to it.
But once it lands you can use nightly which would contain the fix and would ship in FF73.
Comment 11•5 years ago
|
||
Comment 12•5 years ago
|
||
bugherder |
Comment 13•5 years ago
|
||
AFAICT this pref is not exposed in preferences and is disabled by default.
Updated•5 years ago
|
Updated•5 years ago
|
Comment 15•5 years ago
|
||
Firefox 73 - not fixed
Why do you write that in version 73 this problem is fixed? With the mouse wheel, the tabs scroll only visible, when it goes abroad, the tabs are not selected through the mouse wheel.
Description
•