Skip to content

Quieten the crease labels, and say which table the rank comes from - #15

Merged
dgoodenough merged 1 commit into
mainfrom
claude/mirror-split-games-view-jsy6v0
Sep 16, 2026
Merged

dgoodenough merged 1 commit into
mainfrom
claude/mirror-split-games-view-jsy6v0

Conversation

@dgoodenough

Copy link
Copy Markdown
Owner

Two follow-ups to #14.

The crease labels are a caption now

MEN'S and WOMEN'S were set at a fifteenth of the grid's width — a headline stamped across the data rather than a note about the fold. They now use the panel's own label size and colour (--ink-2, 10–20px rather than 12–58px), sit further out towards the corners, and are haloed thinly enough to survive both the red field and the green ramp without reading as a badge.

render_hero.py follows, so the README image is quiet in the same way. The share card is unchanged — it never drew them.

Before After
size clamp(12, L/15, 58) clamp(10, L/40, 20)
colour --ink at 62% --ink-2
halo size / 2.5 size / 4
position 70% / 28% 74% / 24%

Rank by

The sheet carries two FIFA rankings, and which of them orders it is not a question the sheet can answer for you. A Rank by select under Sort now reads:

It greys out (control and label) under Total matches played and Alphabetical, the two orders that read no ranking table — an enabled control that does nothing is the thing this panel already goes out of its way to avoid.

An average is an average only where there are two numbers to take it of, so a team the chosen table does not rank sorts last — as the ~14 members with no women's ranking already did. The default always has a number for every team.

The rank printed beside a team in the panel moves with the setting, names it on hover, and lands between the tables often enough to want a decimal (Australia is #21.5 on the average). State is in the URL as ?rank=men|women|avg.

Testing

Verified in Chromium:

  • All four bases produce the right AFC head and tail — e.g. Japan 17/5 leads on every one; Iran (22/68) is second on men's and sixth on best; Oman, Qatar and Yemen (no women's rank) fall to the end of their confederation on both women's and average.
  • Australia's averaged rank renders as #21.5, and the chip's tooltip names the table.
  • The select disables and dims under matches and alpha, enables under rank and confed.
  • ?rank=avg&sort=rank round-trips; ?fold=1&rank=women carries both.
  • Labels read at their new size in dark and light, over the red field and with Never-played off (green ramp); hero re-rendered and eyeballed at 1:1.
  • Folding still clean after a re-sort — no leaked paint state, no orphaned sheets, no console errors.

python -m unittest discover -s tests — 37 passed.

🤖 Generated with Claude Code

https://claude.ai/code/session_01Ktg1PkqJw6sBz1Wh5qyDsh


Generated by Claude Code

MEN'S and WOMEN'S were set at a fifteenth of the grid's width, which made
them a headline stamped across the data rather than a note about the fold.
They are a caption now: the panel's own label size and colour, pushed out
towards the corners, haloed thinly enough to survive the red field and the
green ramp without reading as a badge. render_hero.py follows, so the README
image is quiet in the same way. The share card is unchanged — it never drew
them.

The sheet also carries two FIFA rankings, and which of them orders it is not
a question the sheet can answer for you. *Rank by* now reads the men's table,
the women's, the average of the two, or — the default, and what the fold
shipped with — whichever of a team's two ranks is better. It sits under Sort
and greys out over the two orders that read no ranking table at all.

An average is an average only where there are two numbers to take it of, so a
team the chosen table does not rank sorts last, as the women's-only teams
already did. The rank printed beside a team in the panel moves with the
setting and names it on hover, and lands between the tables often enough to
want a decimal.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Ktg1PkqJw6sBz1Wh5qyDsh
@dgoodenough
dgoodenough merged commit 7d52b8c into main Sep 16, 2026
1 check 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.

2 participants