fix(site): three fidelity gaps from #102 sign-off #164
No reviewers
Labels
No labels
build-dist
priority
P1
priority
P2
priority
P3
priority
P4
bug
duplicate
enhancement
help wanted
invalid
question
wontfix
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set
Reference
Toilville/toilville-design!164
Loading…
Reference in a new issue
No description provided.
Delete branch "fix/site-fidelity-gaps-102"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Closes #102 (pending Ville's confirmation there — see below).
What changed
color-mix22% of--deep) the canvas gives them, on a white band — matching "Who we are"'s panel formula. The seam-chrome default (a white card on a tinted band) was right for Rituals' own seam section, but wrong for this one; the adapter now tells them apart by surface + chrome instead of guessing one tone for all of them."news"tone narrows the headline measure to the design's own760px/clamp(38px,5vw,64px), instead of inheriting Rituals'900px/clamp(40px,5.4vw,74px). The wider measure was fitting fewer characters per line, which is why it wrapped to 2 lines instead of the canvas's 3 — a wrong column width, not a line-count rule.Regenerated via
python3 tools/wp_theme_ingestor/build_surfaces.py build;build_surfaces.py checkis clean and the pipeline invariant tests pass (143 passed, 1 skipped — withTOILVILLE_RITUALS_DIRpointed at the sibling checkout).Not in this PR
product-cardhas had alockupattribute and the SVGs have been inblocks/_shared/assets/for a while. Flagging that correction on #102 directly, no code change here.toilville_ritualsPR (Toilville/toilville_rituals#459).Merge order
Merge Toilville/toilville_rituals#459 first. This PR's new
"deep"/"news"tone values are inert no-ops on the WordPress side until that PR's block CSS lands.