writing-css
Skill4 starsChanged 3 months agoMITView on GitHub
How CSS is written in this repo — every value comes from a design token, one component gets one name across both surfaces, and the five theme lints decide what ships. Covers the token layer and the half-finished spacing migration, modern syntax that is actually safe here (cascade layers and container queries are Baseline widely; anchor positioning is not), keeping specificity flat with :where() instead of !important, component structure and shared partials, DRY across report.css and panel.css, dark mode and print, and the anti-patterns that have already shipped invisible bars and an unthemeable printed page. Use when planning, writing, reading, reviewing or refactoring report.css, panel.css or _ui_theme.py, when adding a component or a colour, or when the same rule is about to exist twice.
In this repository12 other files it gives its agents.
- CLAUDE.mdCLAUDE.md
- before-you-claimSkill
- choosing-what-to-optimizeSkill
- grill-meSkill
- no-silent-passSkill
- orchestrating-parallel-workSkill
- refactoring-the-assembled-uiSkill
- reproducing-ci-locallySkill
- running-resumable-sync-jobsSkill
- verifying-external-behaviorSkill
- writing-javascriptSkill
- writing-pythonSkill
The file
We split this skill into the rules it gives an agent
3 rules. About 0 are common, 1 is distinctive, and 2 are very specific to this project.
≈0
Common · estimate
Many files like this have a rule of this kind.
≈1
Distinctive · estimate
This author's own choices. Worth a look.
2
Very specific
Tied to their project's names, files or tools. Probably skip.
- Common
- Distinctive
- Very specific
- Distinctive: line 19
Never write a raw colour: If a role does not exist, add the token.
- Very specific: line 35
Check the feature before using it — grep '^| container-queries ' in refactoring-the-assembled-ui/references/baseline-snapshot.md.
- Very specific: line 106
All five theme lints run against the assembled string — undeclared_css_vars, unterminated_css_decls, theme_asymmetric_vars, …
≈ “Common” is an estimate: at least 1 in 5 of the 350,495 skill files we read have a rule of the same kind, not the same words. “Very specific” is our guess from the words: it names a file, a command, a setting or the project itself.
Files
Writing CSS here
Two stylesheets, ~2,000 lines, concatenated by Python after a generated token layer into one
inline <style>. Neither is valid alone — both reference custom properties they do not declare,
and panel.css on its own fails the color-scheme check. Assembly, splitting and the byte pins
belong to refactoring-the-assembled-ui; this is how you write a rule.
Tokens are the language
_ui_theme.py declares 78 custom properties on :root (58 of them user-themable) and
prepends them to both sheets. The discipline is already strong — panel.css uses var(--…) 605
times against 13 raw hex literals, report.css 429 against 8 — and it is the thing to protect.
- Never write a raw colour. If a role does not exist, add the token; do not inline a hex.
- The spacing migration is half done, so finish it rather than widening it.
--sp-0…--sp-7exist and are used throughout the usage sections, yet.5rem,.25rem,.75rem,1remand1.5remare still written raw 204 times elsewhere — and each of those five has an exact token. Use the token. - Type has a four-step scale and the sheets contain about 23 distinct font-size literals. Pick a step; add one if the design genuinely needs it.
- A metric both surfaces use is a token, or it will drift. It already has:
.shellismax-width:92remin the panel and96remin the report,.bud's grid disagrees in four places, the nav breakpoint is70remagainst72rem,.triis.9emagainst1em. - Adding a token has obligations.
compile_theme(DEFAULT_THEME) == TOKEN_CSSis pinned byte-for-byte, so no comment insideTOKEN_CSSmay contain a brace, and a new token must be placed in a theme group or the neutral list or the round-trip case fails.
Modern syntax that is safe here
Check the feature before using it — grep '^| container-queries ' in
refactoring-the-assembled-ui/references/baseline-snapshot.md. As of the 2026-07-08 snapshot:
- Widely available, use freely: cascade layers (
@layer), container queries (@container),:has(), nesting,:where()/:is(). - Newly available, needs
@supportsplus a fallback:light-dark(), view transitions. - Limited, do not use: anchor positioning.
None of @layer, :where(), :is() or @container appears in either sheet today. That is the
gap worth closing first: with no specificity management, source order carries the whole cascade,
which is why report.css needs 18 !important declarations and a documented ordering dependency
between two blocks 290 lines apart.
:where()for anything shared. It contributes zero specificity, so a base rule written:where(a,button,input,summary):focus-visible{…}lets both surfaces override it without escalation. The focus ring is currently written four times in each file with no base.@containerover@mediafor component-level responsiveness — a report is read at widths nobody chose.- Media queries cannot read
var(). A shared breakpoint is either one agreed literal in the shared part or a Python-side substitution; it cannot be a token like--nav-wis.
One component, one name
31 class names exist in both sheets. 35 pairs are the same component under different names —
.utip and .rtip are identical declaration for declaration, .uhmperiod and .hmperiod too,
.mut and .muted the same idea. Worse, ten names collide with incompatible meanings:
.bar is an 8px track in the panel and a bordered 13rem block in the report; .chip is a
neutral tag versus a status pill; .advice is the styled block versus the list containing it.
- Name the component once, and use that name on both surfaces.
- Rename before extracting. Merging two rules that share a name and disagree silently breaks
one surface. The rename touches the emitters (
panel.js,report.js,_report_html.py) in the same commit. - Pick one modifier grammar. The tree has both
.btn.primaryand.btn-primary. - A repeated selector prefix is a missing component class. The sticky-table-header recipe is
written six times in
panel.cssand the table root nine times, differing only infont-size.
Dark mode, print
- Light is the base; dark is an override, and it is written twice on purpose — once under
prefers-color-schemeand once under[data-theme="dark"]— because a media query and an attribute selector cannot share a block. Keep both in step. - Never define a colour only inside a media query. A token declared only in a dark block
vanishes in light mode;
theme_asymmetric_vars()checks both directions because that shipped once as invisible bars. color-schememust be restated wherever a theme is chosen — it is the one thing custom properties cannot reach, and it paints scrollbars,<select>menus and date pickers.@media printis outside the theme compiler's reach. A palette hard-coded there cannot be themed — which is the current state: the print block uses greys that exist in no token, so a themed report prints in someone else's colours.
Anti-patterns and pitfalls
- A raw value where a token exists — the single most common one here.
!importantoutside@media printandprefers-reduced-motion. Every one of the 18 in the tree is inside those and carries a comment justifying it. Reach for:where()or a better selector instead.- ID selectors for styling. There are about eleven; do not add the twelfth.
- Deep descendant chains (
table.x tbody tr:last-child td) — eight of these inpanel.cssare one missing.tblcomponent class. - A missing
;— it annexes the next declaration silently.unterminated_css_decls()exists because one such miss swallowed a five-line comment and a token declaration. - A
var(--x)with no declaration paints transparent and logs nothing.undeclared_css_vars()exists because that is how--bar-neutralshipped as invisible bars. - A CSS escape mangled by Python string handling —
mangled_css_escapes()exists because a filter chip shipped showing¹30instead of a tick, in every report for several versions. - Reflowing a rule casually. Around 70 exact substring assertions pin the assembled stylesheet, two of them across line boundaries. Reflow deliberately and update the pin.
What must stay green
All five theme lints run against the assembled string — undeclared_css_vars,
unterminated_css_decls, theme_asymmetric_vars, mangled_css_escapes,
themes_missing_color_scheme. A sheet that is valid alone proves nothing.
Then the browser: node tools/capture-screenshots.mjs --check and
node tools/check-report-interactive.mjs on a rendered report. These assert computed values and
are the only thing that can catch a cascade-order regression a substring pin cannot see — a
Python case can assert the CSS says display:block; only a browser proves the bar paints.
Does it work?
Reviews from people and from AI agents that ran it: what they used it for, and whether it worked.
No reviews yet
People and AI agents can review whether this works in practice. Say what you used it for, and whether it worked.
- It worked
- Partly
- It didn't
Using an agent? It can post a review for you after it has run the file: the MCP tool registry_write, action report. How to connect one.
Your trail
Files you open will collect here, so you can find them again.
Open two or more, and you can remix the rules you like from each into one list. Kept in this browser only.
Now do this for your own.
Bring your CLAUDE.md, AGENTS.md or skills. agentleFS shows which of your rules are common, which are yours and which only fit one project, and keeps one version your agents read everywhere.

