Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@bdach
Copy link
Collaborator

@bdach bdach commented Dec 17, 2025

Screen.Recording.2025-12-17.at.12.22.00.mov

First part of #36039, see also #33784.

The goal is to add this back to the set panels too, but that one is more complicated than this if you can believe it (because it requires being able to tell which other difficulties of the set are filtered out to fade them out which is not super easy given new song select structure). I also foresee the display logic to vary significantly there.

Of note, for this to work in beatmaps-split-apart mode, this requires a change of behaviour in the filtering logic. Old song select when given a "selected beatmap set" would include that in results regardless of the filter, but that doesn't work for beatmaps-split-apart for reasons that are hopefully obvious, so this changes the behaviour to entirely bypass the filter and just show the set. Unsure how angry will people be with that.

Also of note, when in the scoped mode, altering any filter criteria will dismiss the scope.

… apart

First part of ppy#36039, see also
ppy#33784.

The goal is to add this back to the set panels too, but that one is more
complicated than this if you can believe it (because it requires being
able to tell which other difficulties of the set are filtered out to
fade them out). I also foresee the display logic to vary significantly
there.

Of note, for this to work in beatmaps-split-apart mode, this requires a
change of behaviour in the filtering logic. Old song select when given a
"selected beatmap set" would *include* that in results regardless of the
filter, but that doesn't work for beatmaps-split-apart for reasons that
are hopefully obvious, so this changes the behaviour to *entirely
bypass* the filter and just show the set. Unsure how angry will people
be with that.

Also of note, when in the scoped mode, altering any filter criteria will
dismiss the scope.
@bdach bdach requested a review from peppy December 17, 2025 11:30
@bdach bdach self-assigned this Dec 17, 2025
@bdach bdach moved this from Inbox to Pending Review in osu! untitled project Dec 17, 2025
@peppy
Copy link
Member

peppy commented Dec 18, 2025

Input handling edge case:

osu.2025-12-18.at.13.10.08.mp4

Also, after clicking successfully, it seems like all the little difficulty markers disappear briefly:

osu.2025-12-18.at.13.11.46.mp4

@bdach
Copy link
Collaborator Author

bdach commented Dec 18, 2025

Input handling edge case:

that'll be a failing of OsuAnimatedButton, unsure of what you want me to do about it

Also, after clicking successfully, it seems like all the little difficulty markers disappear briefly:

is that frame by frame playback? how am I supposed to check this?

@peppy
Copy link
Member

peppy commented Dec 18, 2025

I wonder if we should force the beatmap set display into grouped mode (ie. temporarily turn off user sort/group modes). This would allow showing the set header (which also allows operations like deleting the full beatmap set, people have trouble finding this so giving them one other method of doing so would not hurt IMO).

@peppy
Copy link
Member

peppy commented Dec 18, 2025

that'll be a failing of OsuAnimatedButton, unsure of what you want me to do about it

I think the best way is to make the button itself (either visibly or virtually) larger. I was going to mention that it's a very small hit target already, so increasing the size will probably solve two issues.

A good starting point would be to match the height of the pill to the left. It's a small adjustment but should help considerably:

osu! 2025-12-18 at 13 30 54

That said, even if it's an OsuAnimatedButton issues, I'm not sure I'd expect the beatmap panel to handle the OnClick since it shouldn't get the MouseDown authority..

is that frame by frame playback? how am I supposed to check this?

I was seeking through a video yes, but it's very obvious to me in realtime too.

@peppy
Copy link
Member

peppy commented Dec 18, 2025

One more thing: Escape works to exit this display mode, but if you have an active search it also clears that. I don't think the first escape press should clear the search from the textbox in such a case.

@bdach bdach force-pushed the reveal-other-difficulties-thing branch from b4b74dd to 4cce6aa Compare December 19, 2025 12:59
@bdach
Copy link
Collaborator Author

bdach commented Dec 19, 2025

I wonder if we should force the beatmap set display into grouped mode (ie. temporarily turn off user sort/group modes)

I have no strong opinion on this so I'm leaving it be as-is unless I get told to change it.

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

Projects

Status: Pending Review

Development

Successfully merging this pull request may close these issues.

2 participants