From 5b6ff2098d79688385accd1044ed90d6b6466681 Mon Sep 17 00:00:00 2001 From: Pieter Viljoen Date: Thu, 6 Aug 2026 18:08:17 -0700 Subject: [PATCH] Give .gallery-cols-5 the width its own pattern implies The gallery column rules are 1/N widths: 100%, 50%, 33.3333333333%, 25%, 16.666666666%. The five-column case repeated the four-column 25% rather than 20%, which is the shape of a copied line, not a decision. It arrived with the original edit to the theme's extended/blank.css slot and moved verbatim in #40, which claimed to change no rendered byte and so could not correct it. The rendered effect today is none, and the raised finding overstated it. One page passes cols="5", and its gallery is empty: {{< gallery cols="5" >}} {{< /gallery >}} so it emits
with no figures inside, and no figure anywhere on the site is laid out by this rule. The five-per-row claim was not checked against the rendered page before it was repeated. This is therefore a latent fix: it is what the next cols="5" gallery will need, not a defect anyone can see now. The empty shortcode is a content artifact from the WordPress import and is left alone, since content/ is an imported archive that style sweeps do not reach. The rule-level diff of the emitted stylesheet is one line, 25% to 20%. Every built page still differs, because the stylesheet is fingerprinted and each page carries its hash and integrity attribute; per page that link is the only change. URL contract passes 328/328, 778/778, 1012/1012. themes/PaperMod/ is still byte identical to upstream 154d006e. TODO.md loses the Open decisions entry that recorded this, since it is done. Co-Authored-By: Claude Opus 5 (1M context) --- TODO.md | 1 - assets/css/extended/custom.css | 2 +- 2 files changed, 1 insertion(+), 2 deletions(-) diff --git a/TODO.md b/TODO.md index 857856d..da499b2 100644 --- a/TODO.md +++ b/TODO.md @@ -51,7 +51,6 @@ The reference leaf the hub now ships carries one step this repo's deploy does no ## Open decisions - `/robots.txt/` and `/osd.xml/` currently sit in `slugs.map` pointing at `/`. The first would be better pointing at the real `/robots.txt`. -- `.gallery-cols-5 figure` is `width: 25%` where the rule's own 1/N pattern makes it `20%`, so a five-column gallery renders four per row and wraps the fifth. It is a copy of the `cols-4` value, it predates the move of these rules to [`assets/css/extended/custom.css`](./assets/css/extended/custom.css), and one page uses it: [the Flair smart vents post](./content/posts/2022/10/30/installing-flair-smart-vents-to-keep-room-temperatures-balanced.md). Left alone in the move so that change could claim byte-identical rendered output and be verified by diffing the built site; fixing it is a one-value change that must be seen rendered rather than diffed. - Content is capped at a fixed 720px on every screen, because PaperMod's width is four CSS variables with no responsive term and no Hugo parameter. The prose measure is right and should stay; images and galleries inheriting the same cap is the part that costs something on a wide display. The knobs, the override location, and the `--gap` trap are documented under "Customization points" in [`themes/README.md`](./themes/README.md). ## Deliberate deviations from the fleet baseline diff --git a/assets/css/extended/custom.css b/assets/css/extended/custom.css index e25f6ea..4fe3fe5 100644 --- a/assets/css/extended/custom.css +++ b/assets/css/extended/custom.css @@ -55,7 +55,7 @@ body { } .gallery-cols-5 figure { - width: 25%; + width: 20%; } .gallery-cols-6 figure {