Disable switching tabs by scrolling in the tab bar when it doesn't overflow when not part of platform convention
Categories
(Toolkit :: UI Widgets, defect, P2)
Tracking
()
Tracking | Status | |
---|---|---|
firefox-esr60 | --- | unaffected |
firefox63 | --- | unaffected |
firefox64 | --- | unaffected |
firefox65 | + | verified |
firefox66 | + | verified |
People
(Reporter: yoasif, Assigned: dao)
References
Details
(Keywords: regression)
Attachments
(1 file)
47 bytes,
text/x-phabricator-request
|
RyanVM
:
approval-mozilla-beta+
|
Details | Review |
Assignee | ||
Comment 1•7 years ago
|
||
Reporter | ||
Comment 2•7 years ago
|
||
Updated•7 years ago
|
Comment 4•7 years ago
|
||
Updated•7 years ago
|
Comment 5•7 years ago
|
||
Assignee | ||
Updated•7 years ago
|
Assignee | ||
Comment 6•7 years ago
|
||
Comment 8•6 years ago
|
||
bugherder |
Comment 9•6 years ago
|
||
Thanks for doing this, Dão. Please nominate for Beta approval when you get a chance.
Updated•6 years ago
|
Comment 10•6 years ago
|
||
This is okay to uplift to beta.
Comment 11•6 years ago
|
||
Comment on attachment 9035258 [details]
Bug 1512493 - Make switching tabs by scrolling pref-controlled and off by default. r=stransky
[Beta/Release Uplift Approval Request]
Feature/Bug causing the regression: Bug 1285812
User impact if declined: This makes the swith-tab-on-scroll behavior, as implemented in bug 1285812, opt-in, matching the current behavior in GTK3.
This introduces a new pref - 'toolkit.tabbox.switchByScrolling' - which turns this behavior ON with set to true in about:config or in the distribution-specific preferences.
Is this code covered by automated tests?: No
Has the fix been verified in Nightly?: No
Needs manual test from QE?: No
If yes, steps to reproduce:
List of other uplifts needed: None
Risk to taking this patch: Low
Why is the change risky/not risky? (and alternatives if risky): This updates the behavior to match that of the latest GTK version(s), now gated by a pref, not by introducing a risky change.
String changes made/needed: n/a.
Comment 12•6 years ago
|
||
Comment on attachment 9035258 [details]
Bug 1512493 - Make switching tabs by scrolling pref-controlled and off by default. r=stransky
[Triage Comment]
Per UX and the various bugs discussing this, I agree with putting this behind a pref for now. Approved for 65.0b11.
Comment 13•6 years ago
|
||
bugherder uplift |
Updated•6 years ago
|
Comment 14•6 years ago
|
||
I verified the pref "toolkit.tabbox.switchByScrolling" and it is off by default and the scroll function in tab bar doesn't work only if the pref's value is changed to true. Tests were performed on Firefox Beta 65.0b11 and latest Nightly 66.0a1 (2019-01-14) under Ubuntu 16.04 (x64).
Comment 15•6 years ago
|
||
Works fine on Windows as well (tested with today's nightly and 65.0b11) - very nice! :)
Any plans to add additional prefs to control:
(1) whether to allow changing tabs by scrolling in case of an overflowing tab bar as well
(2) whether to wrap at the end/beginning of the tab bar (ie scrolling past the last tab going back to the first tab)
Previously I was using this workaround which supported both of the things I mentioned above (not sure why (1) worked there tbh):
https://forum.manjaro.org/t/howto-enable-tab-switching-in-firefox-using-mouse-wheel/39954
I wouldn't mind submitting a patch for this (especially (2), since (1) looks trickier and more risky to break something unrelated since it seems to be handled by code in scrollbox which may or may not be used elsewhere) if it has a decent chance to get merged.
Description
•