Desktop menu cleanup v2 / UI Guidelines#10637
Draft
MostCromulent wants to merge 2 commits intoCard-Forge:masterfrom
Draft
Desktop menu cleanup v2 / UI Guidelines#10637MostCromulent wants to merge 2 commits intoCard-Forge:masterfrom
MostCromulent wants to merge 2 commits intoCard-Forge:masterfrom
Conversation
65ce4dc to
32bcf8e
Compare
Reorganize the match-screen menu, group layout-file operations together, add a UI Guidelines doc, and tidy shared menu helpers. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
32bcf8e to
5f6af0f
Compare
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Game - how user interacts with game state

Layout - UI layout and theme

Display - in-match display options; how game state is rendered to user

New UI Guidelines doc at
docs/Development/UI-Guidelines.mdcapturing the desktop-vs-mobile conventions and design principles discussed on Discord. Linked from the wiki sidebar.Backend menu-code cleanup:
MenuUtil. Checkbox/radio menu items that previously closed the menu after a click now useMenuUtil.createStayOpenCheckBox/createStayOpenRadioButton(and skinned variants), with the shared mouse-event interception in one private helper. Drops duplicate private factories that lived inLayoutMenu. Theme-skin selection is the one intentional exception (still closes the menu on click).MenuUtil.withMatchUI(Consumer<VMatchUI>)to consolidate theSingletons.getControl().getCurrentScreen().getView()-cast-to-VMatchUIpattern. Replaces 5 cast sites inLayoutMenuand 1 inForgeMenu.showIconsconstant and the conditionalsetIconcalls it gated inGameMenu,DisplayMenu, andLayoutMenu. Also dropped the dock-style icon plumbing it supported inDisplayMenu(themenuIcontracking, the parent-submenu icon update, and thesetTargetingArcMenuIconhelper).WorkshopFileMenuandDeckFileMenustill use a liveshowIconssetter and are untouched.LayoutMenu.currentScreeninstance field — it's now a local ingetMenu().🤖 Generated with Claude Code