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

Revert "Block input to objects lying under already-hit hitcircles when classic note lock is active" #24745

Merged
merged 2 commits into from
Sep 8, 2023

Conversation

bdach
Copy link
Collaborator

@bdach bdach commented Sep 8, 2023

The goal of the aforementioned change was to handle an edge case discovered during work on note lock, wherein it was determined that on stable hit circles would block input from reaching objects underneath them. However, the change mentioned above did that too hard and caused overlaps to also be blocked even long past a hit circle has been faded out.

Revert the change pending further (and more careful) investigation.

…n classic note lock is active"

This reverts commit 03c61a5.

The goal here was to handle an edge case discovered during work on note
lock, wherein it was determined that on stable hit circles would block
input from reaching objects underneath them. However, the change
mentioned above did that _too_ hard and caused overlaps to also be
blocked even long past a hit circle has been faded out.

Revert the change pending further (and more careful) investigation.
@bdach bdach added ruleset/osu! area:gameplay next release Pull requests which are almost there. We'll aim to get them in the next release, but no guarantees! labels Sep 8, 2023
@bdach bdach requested a review from peppy September 8, 2023 08:19
@bdach bdach self-assigned this Sep 8, 2023
@peppy peppy merged commit f8405f8 into ppy:master Sep 8, 2023
9 of 11 checks passed
@bdach bdach deleted the am-i-fired-yet branch September 8, 2023 08:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:gameplay 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.

Classic mod gameplay is broken
2 participants