Release: merge dev into main#590
Merged
Merged
Conversation
added 7 commits
May 31, 2026 18:59
…HudText helper Migrated all Gym scenes to use the shared Renderer createHudText helper for consistent text rendering across the project. Changes: - GymSceneBase: addLabel now delegates to createHudText from shared Renderer - GymOverlayUiScene: replaced all ad-hoc this.add.text calls with createHudText - GymAudioFeedbackScene: replaced status text and call log with createHudText - GymSaveLoadScene: replaced state text, event log, and placeholder with createHudText - GymTranscriptScene: replaced event log and header with createHudText - GymUndoRedoScene: replaced counter, undo/redo labels, history, and event log with createHudText - GymDeckRngScene: replaced seed text and status label with createHudText - GymTooltipScene: replaced mode label, section headers, card labels, and event log with createHudText - GymHandPileScene: replaced slider value texts and event log with createHudText - GymGraphicsLightingSpikeScene: replaced fallback text, section header, and event log with createHudText - GymGraphicsShaderSpikeScene: replaced status, section header, and event log with createHudText GymSllScene continues to use addLabel (which now uses createHudText internally). The remaining this.add.text in GymTooltipScene is a custom tooltip renderer callback passed to TooltipManager - intentionally game-specific rendering. No visual regressions - createHudText uses the same FONT_FAMILY, fontStyle: bold, and supports the same fontSize override as the previous ad-hoc calls.
…tionButton via FeudalismAdapter Create src/ui/Renderer/adapters/FeudalismAdapter.ts with a createFeudalismActionButton wrapper that matches the original Feudalism visual style (dark green bg, green text, 155x42px, 17px bold font) and adjusts y-positioning (shared helper expects top edge; Feudalism used vertical centre). Update FeudalismRenderer.ts to import and use createFeudalismActionButton from the adapter, removing the private createActionButton method (3 call sites converted). Changes: - src/ui/Renderer/adapters/FeudalismAdapter.ts — New adapter file with createFeudalismActionButton wrapper - example-games/feudalism/scenes/FeudalismRenderer.ts — Import adapter, replace 3 call sites, remove private method Tests: 131 Feudalism tests pass (7 files), build succeeds.
…n to shared createHudText Completes the migration of all Gym scenes to use the shared Renderer createHudText helper, addressing gaps identified in the audit. Changes: - GymRouterScene.ts: migrated 6 this.add.text() calls (title, subtitle, transition toggle, card title, description, open label) to createHudText; removed local FONT_FAMILY const - GymSceneBase.ts: addButton now uses createHudText for text creation instead of this.add.text() with inline fontFamily/monospace All remaining this.add.text usage in Gym scenes is in GymTooltipScene's custom tooltip renderer callback (line 83) which is intentionally game-specific rendering passed to TooltipManager -- not a duplicated pattern.
- Create LostCitiesAdapter.ts with shared helper re-exports and game-specific wrappers (createLcHudText, createLcMenuButton). - Migrate LostCitiesOverlayManager to use createLcHudText instead of direct this.scene.add.text() calls for overlay titles, headers, data rows, and totals. - Use createLcMenuButton for the menu button in match summary overlay. - Re-export createOverlayButton and dismissOverlay from adapter for single-import convenience.
… DPR-aware placeholder canvas and add browser test for AI out-of-turn play
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
1 participant
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.
Automated release created by ship skill. Associated work item: CG-0MPUGY4TJ000R0C9