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

Make every button in the lobby menu toggable and close windows #30059

Closed
wants to merge 3 commits into from

Conversation

Winkarst-cpu
Copy link
Contributor

@Winkarst-cpu Winkarst-cpu commented Jul 15, 2024

About the PR

Made every button in the lobby menu toggable, and also made all windows close when the lobby menu is closed (except the AHelp window).

Why / Balance

Fixes #29450, fixes #28855. It's much better to make each button toggable and close all windows when the lobby menu is closed.

Technical details

Made every button in the lobby menu to have ToggleMode set to true, and added code to close windows when the lobby menu is closed.

Media

  • Before
2024-07-15.09-54-22.mp4
  • After
2024-07-15.09-47-00.mp4
  • I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

Changelog

🆑

  • tweak: Now buttons in the lobby menu are toggable and all windows close when the lobby menu is closed.

@github-actions github-actions bot added the Changes: UI Changes: Might require knowledge of UI design or code. label Jul 15, 2024
@Winkarst-cpu Winkarst-cpu changed the title Update main menu Make every button in the main menu toggable Jul 15, 2024
@Winkarst-cpu Winkarst-cpu changed the title Make every button in the main menu toggable Make every button in the main menu toggable and close windows Jul 15, 2024
@Winkarst-cpu Winkarst-cpu changed the title Make every button in the main menu toggable and close windows Make every button in the main lobby toggable and close windows Jul 15, 2024
@Winkarst-cpu Winkarst-cpu changed the title Make every button in the main lobby toggable and close windows Make every button in the lobby menu toggable and close windows Jul 15, 2024
@Winkarst-cpu Winkarst-cpu marked this pull request as draft August 27, 2024 08:42
@SlamBamActionman SlamBamActionman added the S: Untriaged Status: Indicates an item has not been triaged and doesn't have appropriate labels. label Nov 14, 2024
@eoineoineoin
Copy link
Member

Are you going to come back to this? Would be good if you could fix #30059, too, since it's so related.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: UI Changes: Might require knowledge of UI design or code. S: Untriaged Status: Indicates an item has not been triaged and doesn't have appropriate labels.
Projects
None yet
3 participants