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

Fix backwards z-ordering of fruits in juice streams and banana showers #26336

Merged
merged 2 commits into from
Jan 3, 2024

Conversation

bdach
Copy link
Collaborator

@bdach bdach commented Jan 2, 2024

Closes #25827.

master this PR
osu_2024-01-02_12-00-07 osu_2024-01-02_12-01-24
osu_2024-01-02_12-00-30 osu_2024-01-02_12-01-39

The logic cannot be easily abstracted out (because CompareReverseChildID() is protected and non-static on CompositeDrawable), so a local copy was applied instead.

No testing as any testing would have been purely visual anyways.

Closes ppy#25827.

The logic cannot be easily abstracted out (because
`CompareReverseChildID()` is protected and non-static on
`CompositeDrawable`), so a local copy was applied instead.

No testing as any testing would have been purely visual anyways.
@peppy peppy self-requested a review January 3, 2024 03:53
@peppy peppy merged commit f2e0c87 into ppy:master Jan 3, 2024
@bdach bdach deleted the catch-fruit-layering branch January 3, 2024 07:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Fruits in osu!catch have inconsistent layering
2 participants