Skip to content

Unit cards: portraits at the game's own scale - #8

Merged
Perwahl merged 2 commits into
mainfrom
card-portraits
Oct 7, 2026
Merged

Perwahl merged 2 commits into
mainfrom
card-portraits

Conversation

@Perwahl

@Perwahl Perwahl commented Oct 7, 2026 •

Copy link
Copy Markdown
Owner

Follow-up to #7, where portraits were scaled to fit their card. That made every model look the same size.

  • Whole cells: portraits are now the whole cell from the game's sheet (re-exported by Perwahl/FBS#114). How much of its cell a model fills is how the art shows its size.
  • One frame for every unit: all portraits are drawn in the same square frame, so relative sizes match the game. A Newt is smaller than a Firelizard.
  • largeImage: these units are drawn 1.2×, as on the in-game unit card (UIUnitCard.Flash). The column is sized so they fit without clipping.
  • Zero limits: a unit-count limit of 0 reads "not allowed", e.g. Dreadmaw in Warband and Border Patrol.

The first commit's per-card sizing script is removed by the second.

🤖 Generated with Claude Code

Perwahl and others added 2 commits October 7, 2026 22:09
The art column is now the card's height times the portrait's aspect
(9-17rem), measured once the page has laid out, so tall commander cards
show a large figure again and wide monsters get a wide column, with no
cropping. A unit-count limit of 0 reads 'not allowed'.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Portraits are now whole cells from the game's sheet (re-exported), drawn
in one square frame for every unit, so a model's size in its cell shows
its size against other units - a Newt is smaller than a Firelizard.
largeImage units are drawn 1.2x as on the in-game card, and the column
is sized so they fit. Replaces the per-card sizing script.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@Perwahl Perwahl changed the title Unit cards: portrait column sized to the card and the portrait Unit cards: portraits at the game's own scale Oct 7, 2026
@Perwahl
Perwahl merged commit 15c90be into main Oct 7, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant