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

Inability to adjust UI scale key bindings is not intuitive #31360

Open
abraker95 opened this issue Dec 31, 2024 · 4 comments
Open

Inability to adjust UI scale key bindings is not intuitive #31360

abraker95 opened this issue Dec 31, 2024 · 4 comments

Comments

@abraker95
Copy link

Type

Game behaviour

Bug description

I want to change mania scroll speed key bindings to ctrl+= and ctrl+- like I have them in stable. It lets me even though that is mapped to UI scale adjustment. I don't see the UI scale adjustment binding config in settings, and it is taking precedence over the mania scroll speed bindings I set.

Expected behavior:

  • Selection should warn that ctrl+= and ctrl+- key bindings have something already set it
  • Ability to change UI scale adjustment key bindings

Screenshots or videos

No option to change UI scale keybinding when searching for it:
osu_2024-12-30_21-23-17

I was able to set key bindings for scollspeed to ctrl+- and ctrl+= despite that being mapped to UI scaling adjustment
osu_2024-12-30_21-25-01

Version

2024.1224.1-lazer

Logs

compressed-logs.zip

@bdach
Copy link
Collaborator

bdach commented Dec 31, 2024

The scale keybinds are not adjustable because they are bound to "platform actions" added in ppy/osu-framework#4816 which are not supposed to rebindable. This was pointed out during review of that series of changes but the resolution ended up being moving the scroll speed defaults to different buttons.

@Joehuu @peppy please advise how to proceed

@frenzibyte
Copy link
Member

Adding my two cents here, binding a game-wide action such as changing UI scale to a common modifier such as Control feels odd in itself. I would personally be in favour of defining custom actions for this and map it by default to Alt+Plus and Alt+Minus (or any other less-common modifier).

@Joehuu
Copy link
Member

Joehuu commented Jan 1, 2025

When I made those PRs, I was following what browsers and other apps like Discord that didn't allow rebinding. I totally disregarded games.

After looking for games with UI scale, I can only find Factorio. That game defaults to only Ctrl + Numpad +/-/0 (not number row) and is rebindable.

So I would make them rebindable. Unsure of the default changing.

@peppy
Copy link
Member

peppy commented Jan 2, 2025

At very least, user configured keys may want to override platform keys. If that's not easy to achieve (or not the global bheaviour we want) then redefining these keys locally is probably fine. Just keep in mind they should have different defaults per platform, maybe sourced from the platform bindings' defaults.

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

No branches or pull requests

5 participants