osu icon indicating copy to clipboard operation
osu copied to clipboard

ctrl+up/down hotkey does not work on the multiplayer and daily challenge screens

Open tovc opened this issue 1 year ago • 2 comments

Type

Game behaviour

Bug description

doesn't seem like there's any reason that it shouldn't work. I guess assuming DT is disabled as a freemod in multiplayer is correct most of the time? this seems somewhat reasonable, if a bit annoying in multiplayer, but in the daily challenge, DT is always enabled, so it just feels wrong

Screenshots or videos

No response

Version

2024.817.0-lazer

Logs

compressed-logs.zip

tovc avatar Aug 18 '24 12:08 tovc

Seems valid. This would mean adding ModSpeedHotkeyHandlers to two more screens, at which point I'd hope the key binding handling could be encapsulated within that class.

@bdach What was the reason for not doing this in the first place? On an initial check it looks like the only valid reason would be the pre-check on current screen at song select:

https://github.com/ppy/osu/blob/b567ab2a3923d6579cf23849447546e704a621fa/osu.Game/Screens/Select/SongSelect.cs#L1024

peppy avatar Aug 19 '24 06:08 peppy

The primary reason for not doing it was forgetting that multiplayer exists and therefore not caring what was being handled where.

Do note however that adding that in will require carve-outs for checking if rate adjust is even allowable in a given multiplayer context.

bdach avatar Aug 19 '24 06:08 bdach