v1 audit fixes: dead tokens, stale docs tables, blog corrections#49
Merged
Conversation
Framework and site CSS: - Remove the orphan --meter-color token (nothing consumed it; the meter paints with --meter-color-track/low/med/high), noted in the changelog. - Remove the dead radius-demo rule referencing the nonexistent --radius-drawn token (the docs table has 6 rows, the CSS styled 7). Docs: - tokens: fix the heading-sizes table (every row was one step smaller than the actual clamps), document the new --highlight-500 token, rename the font stack row to the real token name (mono), fix the font stack notice copy. - media-queries: document the user-preference and device-capability queries (OSdark, motionOK, touch, forcedColors, etc.), fix the mangled "(width <section 768px)" cell, note -phone availability. - start: align the states example with the block_element syntax (form_input/form_submit), point the preset-env mention at the right link, fix typos (Targeting, properties). - components/start: correct the zero-JavaScript exceptions list (hero slideshow, TOC scrollspy, read progress bar fallback) and the copy about per-page file tables. - featuregrid: document FeatureItem's titleClass/iconClass props. - Bump lastUpdate on every page with content changes. Blog: - structure post: layer list updated for v1 (Pages layer removed, Theme layer added). - 1.0 post and changelog: component count corrected to 26. - colors post: replace the nonexistent br-sm helper with an inline border-radius. - select-mask post: fix the drifted GitHub line anchor. - BEM post: punctuation cleanup. Also clarifies the em dash rule in CLAUDE.md: legitimate typographic uses (ranges, quote attribution) are fine; sentence-construction dashes are not. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
✅ Deploy Preview for mcssdev ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
broke on commit 43120ec ("Hero updates", 2 Jul 2025)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.

Final pre-tag audit fixes for v1.0. Everything from the audit is addressed except the display-tracking TODO in settings.tokens.css, which stays by request.
CSS bugs (same class as the recently deleted tokens)
--meter-colortoken from elements.form.css: nothing ever consumed it (the meter paints with--meter-color-track/low/med/high). Recorded in the changelog next to the--layout-aside-background-colorremoval. dist/ regenerated by the pre-commit hook.--radius-drawntoken (the tokens page table has 6 rows; the CSS styled a 7th).Docs
heading-1isdisplay-md → display-xl, etc.). Documented the new--highlight-500token with a demo swatch and its relationship to--marker-color. Renamed the font-stack rowmonospaceto the real token namemono. Fixed the confusing font-stack notice ("wether", two things called "the system default").--OSdark/--OSlight,--motionOK,--touch/--mouse/--stylus/--pointer,--forcedColors,--HDcolor, etc.), fixed the mangled(width <section 768px)table cell, and noted that-phonevariants stop atlg.block_elementsyntax it sits under (form_input/form_submitinstead offormInput/formSubmit), the postcss-preset-env mention now links to preset-env instead of postcss-mixins, and two typos are fixed (Targeting, properties).titleClass/iconClassprops (matching how section documents its equivalents).lastUpdatebumped on every page whose content changed; tokens and media-queries also bumped to version 1 since they now reflect the v1 framework.Blog
br-smhelper that doesn't exist; radius moved into the inline style.House style
h1–h6, quote attribution, table placeholders) are fine; sentence-construction dashes are not. Theh1–h6range in the elements docs was tidied to a closed-up en dash under the same rule.Verification
npm run build:css: dist/ regenerated cleanly (meter token gone from dist, layer invariants intact).npm run lint:css: clean.npm run build: 110 pages, no errors; corrected tables, new sections, and class fixes confirmed in the built HTML.npm run check: 0 errors, 0 warnings.🤖 Generated with Claude Code