Skip to content

docs: update design token examples in CONTRIBUTING to match redesign - #70

Merged
royalpinto007 merged 1 commit into
AgentPostmortem:mainfrom
waterlemonnn:docs/fix-contributing-tokens
Aug 7, 2026
Merged

docs: update design token examples in CONTRIBUTING to match redesign#70
royalpinto007 merged 1 commit into
AgentPostmortem:mainfrom
waterlemonnn:docs/fix-contributing-tokens

Conversation

@waterlemonnn

Copy link
Copy Markdown
Contributor

Closes #34.

The UI / bug fixes section pointed contributors at the pre-redesign accent-red / accent-red-muted tokens, which don't exist in tailwind.config.ts anymore, and used unprefixed class names (bg-canvas, text-tertiary) that don't render as-is.

  • Points at the current amber accent set (accent, accent-soft, accent-strong) and writes the examples the way contributors actually type them (bg-bg-canvas, text-text-primary, border-border-default), matching how components/ui/Card.tsx and Button.tsx use them.
  • Calls out the sev-* severity scale as separate from the brand accent, and carries over the never-colour-alone rule from the doc comment on SEVERITY_STYLES in lib/constants/severity.ts.
  • Drops the WCAG contrast sentence rather than inventing a replacement pairing, per the issue.

npx prettier --check CONTRIBUTING.md passes. Repo-wide npm run format/prettier --check . still flags ~145 unrelated files on this checkout (confirmed pre-existing on a clean main before this change, looks like a CRLF line-ending thing on Windows checkouts), so I only touched the one file this issue is about.

The UI/bug fixes section still referenced the pre-redesign accent-red
tokens and unprefixed class names. Point at the current amber accent
and sev-* severity scale instead, written as the utilities contributors
actually type, and carry over the never-colour-alone rule from
lib/constants/severity.ts. Drops the WCAG pairing claim rather than
inventing an unverified replacement.
@royalpinto007
royalpinto007 merged commit 37d19dc into AgentPostmortem:main Aug 7, 2026
11 checks passed
@github-actions

github-actions Bot commented Aug 7, 2026

Copy link
Copy Markdown

Merged. Thanks a lot, @waterlemonnn.

Good, focused change. Exactly the kind of thing agentpostmortem needs.

If agentpostmortem is useful to you, a star goes a long way.

Happy to see more from you whenever you have the time.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CONTRIBUTING points at design tokens that no longer exist after the redesign

2 participants