From 6a9856b5900daa96b2ec4e41a7255743ae38c80b Mon Sep 17 00:00:00 2001 From: debont80 Date: Wed, 1 Jul 2026 08:19:41 -0400 Subject: [PATCH] Self-review: drop inert font-size/weight resets from the heading rule MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The UI33 heading reset only needs to zero the UA margin — every heading in the app carries a Fluent typography class that out-specifies this element selector (and there are no raw headings), so the font-size/font-weight `inherit` lines never applied and contradicted the comment. Keep just `margin: 0`. Co-Authored-By: Claude Opus 4.8 --- src/renderer/src/assets/base.css | 10 ++++------ 1 file changed, 4 insertions(+), 6 deletions(-) diff --git a/src/renderer/src/assets/base.css b/src/renderer/src/assets/base.css index ebafcf5..e484fa1 100644 --- a/src/renderer/src/assets/base.css +++ b/src/renderer/src/assets/base.css @@ -10,11 +10,11 @@ } /* - * Reset the UA heading styles (UI33). Titles are rendered as real headings + * Zero the UA heading margin (UI33). Titles are rendered as real headings * (h1/h2) for Narrator heading-navigation, but they carry Fluent's typography - * classes for their visual ramp; those classes out-specify this element - * selector, so only the browser's default heading margin needs zeroing to keep - * the layout identical to the former styled-span titles. + * classes for their visual ramp (font size/weight/line-height); those classes + * out-specify this element selector, so only the browser's default heading + * margin needs removing to keep the layout identical to the former span titles. */ h1, h2, @@ -23,8 +23,6 @@ h4, h5, h6 { margin: 0; - font-size: inherit; - font-weight: inherit; } html,