-
-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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 reverse selection not updating new combo location #27723
Fix reverse selection not updating new combo location #27723
Conversation
Unit tests missing. |
Since it appears that there is no tests for reverse pattern yet, would it be correct to start adding them to TestSceneOsuComposerSelection and create a new one for catch at here ? |
Sure. |
Since the tests got so big i decided to create its own classes for it: TestSceneOsuReverseSelection and TestSceneCatchReverseSelection |
@64ArthurAraujo i've simplfied the implementation of the fix and also significantly simplified the test coverage. please make sure to check those changes so that future changes can take less time to get in. |
This PR fixes #27319 (both catch and standard).
before.mp4
after.mp4