-
-
Notifications
You must be signed in to change notification settings - Fork 521
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add option to use legacy/office-style mouse wheel tab shifting #1058
Add option to use legacy/office-style mouse wheel tab shifting #1058
Conversation
4eeab22
to
6257329
Compare
I did a quick test and it does not behave as office behaves.
If you want to mirror the exact behavior as in office there is more to be done. |
It's not important to me that it works the same way as in office. The important part is to be able to shift tabs with Mousewheel when inside the groups at all, whether it's like Fluent.Ribbon v8.X or office. It needs to be some kind consistent and least-surprising behavior here though when it comes to group scroll/tab scroll/focus.
Good catch.Will try to sort that. I think it's surprising for the user to get group scrolling only when this option is enabled + a control is focused. Better to make this option completely disable group scrolling.
Ok maybe there are different behaviors also in office? I can't scroll with focus on any controls in (Tried O365 Outlook build 2207 16.0.1427). |
b72bff7
to
4d841c9
Compare
Attempted some changes here. There were two changes
Note that potentially the check in 1) above should perhaps be made regardless of whether IsMouseWheelScrollingAllowedEverywhere is true, because the behavior otherwise is this, which feels kind of buggy However at the moment I have not made that change in this branch. The behavior above remains. |
Will merge your changes, but i am not sure if we should stick with Also i am not sure if we should mark every mousewheel event as handled, when we didn't handle it but just want to skip our own processing. Will try to test some things and see how it behaves. Will fix the bug you discovered regarding group scrolling. |
Thanks. Yes as I said the name is pretty terrible but I felt it was most important to keep it similar in prefix to the other option. If they must be renamed then I think both should be, but then they should perhaps instead be converted to a single Enum with 3 or more different modes for behavior instead of multiple booleans. |
Closes #1055
This adds a new boolean option to use the old-style mouse wheel behavior found in Fluent.Ribbon <=v8 and Office.
The name of the new option is
IsMouseWheelScrollingEnabledEverywhere
, (and it's frankly not a good name...)When
true
, the mouse wheel events are caught and used to cycle tabs, regardless of where on the ribbon toolbar the wheel event occurs. Wen the new option isfalse
, the old behavior remains: mouse wheel in the tab area can cycle tab (conditional on the oldIsMouseWheelScrollingEnabled
option). In both cases, the same logic will potentially block the tab shifting, such as if a control has focus.If
IsMouseWheelScrollingEnabledEverywhere
is true , then the value ofIsMouseWheelScrollingEnabled
will be ignored.The default for the new option is
false
.