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 playlist room creation screen pushing content on opening dropdowns #19065

Merged
merged 1 commit into from Jul 12, 2022

Conversation

Ludio235
Copy link
Contributor

Fixes duration dropdown menu on playlist creation screen so that it does NOT push the content below down as discussed in issue #18162 by copying the solution in multiplayer creation screen.

@frenzibyte frenzibyte changed the title Fixed inconsistent behavior of dropdowns on playlists/multiplayer creation screens Fix playlist room creation screen pushing content on opening dropdowns Jul 11, 2022
@bdach
Copy link
Collaborator

bdach commented Jul 11, 2022

In the future please submit PRs off topic branches rather than your fork's master branch. Please read the contributing guidelines if you haven't done so yet.

@bdach
Copy link
Collaborator

bdach commented Jul 11, 2022

As for the actual code, looks simple enough as this is reusing the same workaround that multiplayer dropdowns use. I did find this one masking quirk:

2022-07-11.21-31-27.mp4

But I'm not super sure it matters too much and the multiplayer dropdowns do the same thing, so probably fine to ignore?

@bdach bdach requested a review from peppy July 11, 2022 19: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.

None yet

4 participants