You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Added the shared Theme V2 toolbox selected-game status bar for toolbox pages.
Preserved Game Hub ownership of selected-game context and Idea Board selected-game filtering exclusion.
Polished the status bar context display so the left side shows selected game name/purpose and the center shows tool action, save state, validation, warning, or error context.
Kept environment details out of the status bar because the platform banner owns that context.
Added ALFA_001 and ALFA_002 reports plus repo-structured delta ZIP artifacts under tmp/.
Validation
npx playwright test tests/playwright/tools/ToolboxSelectedGameStatusBar.spec.mjs --workers=1 - PASS, 6 passed
rg -n "<style|style=" assets/theme-v2/js/toolbox-status-bar.js assets/theme-v2/css/status.css tests/playwright/tools/ToolboxSelectedGameStatusBar.spec.mjs - PASS, no matches
git diff --cached --check before ALFA_002 commit - PASS
The reason will be displayed to describe this comment to others. Learn more.
Mount the bar on the toolbox clean URL
When the toolbox landing is opened as /toolbox/ or /toolbox (the common directory-index URL), currentPagePath() returns toolbox, so this guard returns before loading the shared status bar. That leaves the toolbox index without the new selected-game/context UI even though /toolbox/index.html gets it; normalize the exact toolbox path to the index page or allow it here.
The reason will be displayed to describe this comment to others. Learn more.
Select the status that changed instead of the last one
When a toolbox page has several data-backed status regions, this always picks the last data-status in DOM order rather than the status that actually changed. For example, Game Journey's note/type/editor status nodes appear before data-journey-target-status, so saving or adding a note updates data-journey-note-status but the shared bar keeps showing the recommended-target status instead of the actual tool action/save message.
Useful? React with 👍 / 👎.
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
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.
Summary
tmp/.Validation
npx playwright test tests/playwright/tools/ToolboxSelectedGameStatusBar.spec.mjs --workers=1- PASS, 6 passedrg -n "<style|style=" assets/theme-v2/js/toolbox-status-bar.js assets/theme-v2/css/status.css tests/playwright/tools/ToolboxSelectedGameStatusBar.spec.mjs- PASS, no matchesgit diff --cached --checkbefore ALFA_002 commit - PASSArtifacts
tmp/PR_26175_ALFA_001-toolbox-selected-game-status-bar_delta.ziptmp/PR_26175_ALFA_002-toolbox-status-bar-context-polish_delta.zipCommits
359d48c503557b13e8