Skip to content

Apply Burcu's design-review feedback to Adorn UI#5099

Merged
lukemelia merged 1 commit into
mainfrom
cs-11313-apply-burcus-feedback-to-adorn-ui
Jun 4, 2026
Merged

Apply Burcu's design-review feedback to Adorn UI#5099
lukemelia merged 1 commit into
mainfrom
cs-11313-apply-burcus-feedback-to-adorn-ui

Conversation

@lukemelia

Copy link
Copy Markdown
Contributor

Refactors the Adorn components/modifiers and their consumers to follow the design-system conventions, applying each item from Burcu's review.

Conventions applied

  • data-* over class names for DOM targeting — the label-boundary resolver now keys off data-adorn-context instead of closest('.adorn-context'). The class stays for styling; the data attribute is the JS contract.
  • rem instead of px — converted dimensional CSS across the Adorn primitives, the card-search selection trigger, and the card-header utility-menu trigger. SVG-internal coordinates and JS pixel math left in px.
  • Hardcoded colors → semantic tokens — no raw hex in components; all references go through tokens.
  • cn helper for conditional classes — replaced inline {{if}}/{{unless}} class-string concatenation in AdornLabel / AdornSelectChip (and the AdornContext consumer). Emits identical class strings, so behavior and selectors are unchanged.
  • currentColor for SVGs — the themeable check stroke follows color; fixed brand parts reference a token.
  • --boxel-highlight instead of --boxel-teal for the highlight accent, app-wide.

Token consolidation

  • --boxel-dark-teal is now #00da9f (previously #00ebac), so the existing --boxel-highlight-hover carries it rather than introducing a parallel variable. This is an app-wide shift: hover/highlight states (buttons, pills, selects, pickers, multi-select, codemirror, …) move to the slightly deeper teal.
  • Added --boxel-highlight-foreground: #0a2e1c — the readable dark-green foreground for text/icons on a --boxel-highlight surface, following the established <surface> / <surface>-foreground convention. Wired through the Adorn primitives, the card-search selection trigger, and card-header (whose utility-menu trigger previously duplicated the same pattern with hardcoded colors).

Visual impact

  • No change: teal backgrounds and icon colors resolve to the same hex as before.
  • Intended changes: text/icons on highlight surfaces shift from pure black to the dark-green --boxel-highlight-foreground (the editing card-header bar and the selection/utility-menu triggers), and hover/highlight teal deepens slightly via the --boxel-dark-teal value change.

Docs

  • New gts-component-conventions skill capturing these guidelines.
  • The skill plus AGENTS.md now document the content-tag <template>-detection hazards hit during this work, including a previously-undocumented trigger (a backtick-wrapped bracket token inside a template-region comment) and its ESLint-phantom-unused-import symptom.

🤖 Generated with Claude Code

Refactor the Adorn components/modifiers and their consumers to follow the
design-system conventions:

- Target DOM via a `data-adorn-context` attribute instead of the
  `.adorn-context` class
- Convert px to rem across the Adorn CSS
- Replace hardcoded teal/ink hex values with semantic tokens
- Use the `cn` helper for conditional class names
- Use `currentColor` for the themeable parts of inline SVGs
- Use `--boxel-highlight` / `--boxel-highlight-hover` instead of `--boxel-teal`

Consolidate the teal palette: `--boxel-dark-teal` becomes #00da9f (so
`--boxel-highlight-hover` carries it), and add `--boxel-highlight-foreground`
(#0a2e1c) as the readable foreground on highlight surfaces, following the
established `<surface>` / `<surface>-foreground` convention. The new token is
wired through the Adorn primitives, the card-search selection trigger, and the
card-header utility-menu trigger (which previously duplicated the same pattern
with hardcoded colors).

Document these conventions and the content-tag `<template>`-detection hazards
in a new gts-component-conventions skill and in AGENTS.md.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@lukemelia lukemelia requested review from a team and burieberry June 3, 2026 21:40
@lukemelia lukemelia marked this pull request as ready for review June 3, 2026 21:40
@github-actions

github-actions Bot commented Jun 3, 2026

Copy link
Copy Markdown
Contributor

Preview deployments

Host Test Results

    1 files      1 suites   1h 50m 56s ⏱️
2 931 tests 2 916 ✅ 15 💤 0 ❌
2 950 runs  2 935 ✅ 15 💤 0 ❌

Results for commit 88db2a4.

Realm Server Test Results

    1 files      1 suites   10m 0s ⏱️
1 549 tests 1 548 ✅ 1 💤 0 ❌
1 640 runs  1 639 ✅ 1 💤 0 ❌

Results for commit 88db2a4.

@lukemelia

Copy link
Copy Markdown
Contributor Author

@burieberry still more to come based on your feedback, but please review this batch

@burieberry burieberry left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, thanks for updating the skills!

@lukemelia lukemelia merged commit 264a1de into main Jun 4, 2026
80 of 87 checks passed
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.

2 participants