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

Add classic skin sprites for slider tick and slider end misses #26117

Merged
merged 4 commits into from
Dec 25, 2023

Conversation

peppy
Copy link
Member

@peppy peppy commented Dec 25, 2023

Design feedback was taken in #general on discord.


A lot of complaints asking for disable on the new elements. Mostly because it was confusing to just downscale the normal miss.

Now they get their own sprites. And as a result it's skinnable for the people that are really adverse to change. You can skin via slidertickmiss@2x.png and sliderendmiss@2x.png.

CleanShot.2023-12-25.at.08.12.17.mp4
CleanShot.2023-12-25.at.08.13.40.mp4

@peppy peppy added ruleset/osu! next release Pull requests which are almost there. We'll aim to get them in the next release, but no guarantees! and removed size/S labels Dec 25, 2023
@peppy peppy force-pushed the add-slider-tick-end-miss-sprites branch from 91f4cd1 to b1d994b Compare December 25, 2023 08:33
@pull-request-size pull-request-size bot added size/S and removed size/M labels Dec 25, 2023
@bdach bdach self-requested a review December 25, 2023 12:30
bdach
bdach previously requested changes Dec 25, 2023
@bdach bdach dismissed their stale review December 25, 2023 13:30

seems okay, just waiting for the resources package now

@pull-request-size pull-request-size bot added size/M and removed size/S labels Dec 25, 2023
@bdach bdach enabled auto-merge December 25, 2023 13:59
@bdach bdach merged commit dde88bb into ppy:master Dec 25, 2023
@peppy peppy deleted the add-slider-tick-end-miss-sprites branch January 4, 2024 03:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
next release Pull requests which are almost there. We'll aim to get them in the next release, but no guarantees! ruleset/osu! size/M
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants