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

Apply osu! side changes in line with FocusedOverlayContainer.PopIn abstract change #23882

Merged
merged 2 commits into from Jun 18, 2023

Conversation

peppy
Copy link
Sponsor Member

@peppy peppy commented Jun 12, 2023

Have confirmed that behaviour this side feels normal for popovers / overlays. Also, tests pass:

JetBrains Rider 2023-06-12 at 05 14 25

@peppy peppy force-pushed the focused-overlay-container-override-fixes branch from e607f77 to eb31fde Compare June 18, 2023 11:57
@peppy peppy requested a review from a team June 18, 2023 12:23
@peppy peppy added the next release Pull requests which are almost there. We'll aim to get them in the next release, but no guarantees! label Jun 18, 2023
@bdach bdach merged commit 5dc675a into ppy:master Jun 18, 2023
13 of 17 checks passed
@peppy peppy deleted the focused-overlay-container-override-fixes branch June 20, 2023 04:08
andy840119 added a commit to andy840119/karaoke that referenced this pull request Jun 24, 2023
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! size/M type:code-quality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants