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

EZ/HR/TP mods don't change stats color inside mod select overlay #31313

Open
Gabixel opened this issue Dec 27, 2024 · 1 comment
Open

EZ/HR/TP mods don't change stats color inside mod select overlay #31313

Gabixel opened this issue Dec 27, 2024 · 1 comment

Comments

@Gabixel
Copy link
Contributor

Gabixel commented Dec 27, 2024

Type

Cosmetic

Bug description

From my testing, the values don't change color with Easy, Hard Rock and Target Practice

Screenshots or videos

image

It does change color, however, under the carousel wedge:

image

Other mods like Half Time and Double Time seem to update the color fine:

image

Version

2024.1224.1

Logs

n/a

@bdach
Copy link
Collaborator

bdach commented Dec 30, 2024

The way this is currently implemented is such that colours only get used on the mod select screen if there are rate-changing mods active that change the "effective" value of one of these difficulty parameters. HR has nothing to do with rate change which is why this isn't highlighted.

That said this is all over the place because the bars under the wedge show the "not effective" values. Dunno. Needs some executive call on how this should work. Obviously the goal here is parity between the two displays, but which way it should be done is disputable. I'd say we want to show the "effective" values everywhere but always use red/green when the values change from base, not only for rate-change induced differences.

@ppy/team-client would appreciate opinions

@bdach bdach removed their assignment Dec 30, 2024
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

2 participants