Repository navigation
Ice Palace: dot cells, paint, and stashes for the new renderer - #498
Merged
Perlkonig merged 1 commit intoOct 5, 2026
Conversation
Notation
- Cells are written x.y rather than x,y: the front end reads commas in
a move as separating simultaneous moves, so a build of numplayers - 1
pyramids was split into one move per seat. Games saved with comma
cells keep working: moves typed or replayed in the old notation are
read in the new one, and stack entries are rewritten on load.
Rendering
- Every legend glyph is coloured through `paint`, which the renderer
now asks for in place of the deprecated `colour`. Translucency moves
into the fill slot, so outlines stay solid as before.
- The hand and Pool are laid out alike under either board, since both
renderers now lay stashes out with the same step; the 3D-board-only
layouts with placeholders and headroom go.
- In 3D the Pool is shown as nests, each holding at most one pyramid of
each size: the renderer rests a column's pyramids on one base line by
size, so several of a size in one column were drawn as one, and a
spacer among them was an error ("Could not resolve silhouette tier").
- The 3D hand glyph puts the pyramid before its invisible click square,
which the renderer reads for the pyramid's size, so hand pyramids rest
on one base line.
- Black and White pyramids paint their outline and size pips in
whichever of black and white contrasts with the body, so a Black
pyramid shows its pips. The sidebar sends each pyramid as its legend
glyph so the border reaches it too.
Playground
- Status glyphs are sized to the text; the renderer sizes an inline
glyph's svg to 100%, which the wrapper's max-height did not constrain.
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01NSoBhzyAEBBaTsYtVpV8aT
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.
Notation
Rendering
paint, which the renderer now asks for in place of the deprecatedcolour. Translucency moves into the fill slot, so outlines stay solid as before.Playground