Skip to content

Convert vertical scroll to horizontal when "shift" is pressed #9123

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Convert vertical scroll to horizontal when "shift" is pressed #9123

wants to merge 2 commits into from

Conversation

@DexerBR
Copy link
Contributor

@DexerBR DexerBR commented Aug 18, 2025 •
edited
Loading

This PR modifies the scroll behavior to convert vertical scroll events into horizontal ones when the shift key is held. Specifically:

  • scrolldown → scrollright
  • scrollup → scrollleft

This allows to perform horizontal scrolling using the scroll wheel while holding shift.

Depends on:

Maintainer merge checklist

  • Title is descriptive/clear for inclusion in release notes.
  • Applied a Component: xxx label.
  • Applied the api-deprecation or api-break label.
  • Applied the release-highlight label to be highlighted in release notes.
  • Added to the milestone version it was merged into.
  • Unittests are included in PR.
  • Properly documented, including versionadded, versionchanged as needed.

@kuzeyron kuzeyron added the Component: core-widget properties, eventdispatcher, widget.py, animation label Aug 19, 2025
@DexerBR DexerBR force-pushed the scrollview-key-modifier branch from 4cc37d7 to 37d4a43 Compare September 10, 2025 17:15
@DexerBR DexerBR force-pushed the scrollview-key-modifier branch from 37d4a43 to a554029 Compare September 11, 2025 20:38
@DexerBR
Copy link
Contributor Author

DexerBR commented Sep 11, 2025

Update: Since this one is already merged (#9121), this PR is ready for direct testing/review.

@misl6
Copy link
Member

misl6 commented Sep 13, 2025

Maybe we can document that?

I'm trackpad-only since a long time, and I actually forgot about that feature.

@DexerBR
Copy link
Contributor Author

DexerBR commented Sep 28, 2025

Maybe we can document that?

I'm trackpad-only since a long time, and I actually forgot about that feature.

Done 👍

@misl6
Copy link
Member

misl6 commented Sep 30, 2025

Seems a test is failing due to missing modifiers property, can you take a look at it?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

Component: core-widget properties, eventdispatcher, widget.py, animation

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@DexerBR @misl6 @kuzeyron