Skip to content
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 Alt+MouseWheel as a global volume adjust binding #31281

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

peppy
Copy link
Member

@peppy peppy commented Dec 26, 2024

This offers the defaults users have come to expect while also allowing rebinding (if they don't want it) and better compatibility with other global actions (since this is no longer implemented using OnScroll event flow).

Addresses part of #31267.

This offers the defaults users have come to expect while also allowing
rebinding (if they don't want it) and better compatibility with other
global actions (since this is no longer implemented using `OnScroll`
event flow).

Addresses part of ppy#31267.
@bdach
Copy link
Collaborator

bdach commented Dec 26, 2024

relevant test failures, same as #31278 this breaks alt-scroll for timeline zoom in editor. maybe fixable by moving alt-scroll for timeline zoom to a editor hotkey, although that's sorta icky because the whole ZoomableScrollContainer machinery is too generic for its own good so if it ever gets reused outside of editor contexts it'll get stupid (that said it doesn't seem like we do that yet).

Copy link
Collaborator

@bdach bdach left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

as above

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

Successfully merging this pull request may close these issues.

2 participants