Skip to content

Conversation

@rikimasan
Copy link
Contributor

@rikimasan rikimasan commented Dec 18, 2025

Relevant Issues:

Manual Testing Done:

  • Opened multiplayer screen to check I can alt+scroll for volume change.
  • Opened editor to check I can still do things like ctrl+alt+scroll for distance snap change and scroll the timeline.
  • Made sure I can still alt scroll on timeline to adjust zoom

@peppy
Copy link
Member

peppy commented Dec 25, 2025

If we're doing this again, the local implementations should probably be undone, along with understanding the implications.

Basically reverting #31146.

@rikimasan
Copy link
Contributor Author

If we're doing this again, the local implementations should probably be undone, along with understanding the implications.

Basically reverting #31146.

Would you prefer I try to do everything as local implementations or continue with global and remove the existing local implementations?

@peppy
Copy link
Member

peppy commented Dec 29, 2025

Honestly having it global seems like "what most people want". But do keep in mind that it needs to be implemented in a way that doesn't take priority away from other elements, which is one of the reasons we retired this in the first place.

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants