diff --git a/.planning/NOTES.md b/.planning/NOTES.md index 6dd7de0..dd1a3b0 100644 --- a/.planning/NOTES.md +++ b/.planning/NOTES.md @@ -4510,9 +4510,23 @@ stylesheet gives it one, so `font-size: inherit` on a button is a *change*, not a no-op: it took every desktop control from 33x21 to 36x24 by moving them from 13.3px to the shell's 16px. Nothing failed. The only way it surfaced was measuring the baseline by stashing the file -and re-running — which is now what `phone-transport.spec.ts` pins, with -a literal `'33x21'` rather than a range, because a range swallows three -pixels. +and re-running. + +**And the pixel it was first pinned with was the wrong assertion.** The +spec asserted the literal `'33x21'`, measured in Chromium — and WebKit +draws the same button **36x24**, so it failed in CI on a build where +nothing was wrong. A button's box comes from the UA stylesheet when the +author sets nothing, and what each UA sets is its own business. What +must not happen is that *we* set something, so that is what it asserts +now: `min-width` and `min-height` compute to `0px`, and the font-size +still equals that of a bare `