Apply Burcu's design-review feedback to Adorn UI#5099
Merged
Conversation
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>
Contributor
Contributor
Author
|
@burieberry still more to come based on your feedback, but please review this batch |
This was referenced Jun 3, 2026
burieberry
approved these changes
Jun 4, 2026
burieberry
left a comment
Contributor
There was a problem hiding this comment.
Looks good, thanks for updating the skills!
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.
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 offdata-adorn-contextinstead ofclosest('.adorn-context'). The class stays for styling; the data attribute is the JS contract.reminstead ofpx— 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.cnhelper for conditional classes — replaced inline{{if}}/{{unless}}class-string concatenation inAdornLabel/AdornSelectChip(and the AdornContext consumer). Emits identical class strings, so behavior and selectors are unchanged.currentColorfor SVGs — the themeable check stroke followscolor; fixed brand parts reference a token.--boxel-highlightinstead of--boxel-tealfor the highlight accent, app-wide.Token consolidation
--boxel-dark-tealis now#00da9f(previously#00ebac), so the existing--boxel-highlight-hovercarries 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.--boxel-highlight-foreground: #0a2e1c— the readable dark-green foreground for text/icons on a--boxel-highlightsurface, following the established<surface>/<surface>-foregroundconvention. Wired through the Adorn primitives, the card-search selection trigger, andcard-header(whose utility-menu trigger previously duplicated the same pattern with hardcoded colors).Visual impact
--boxel-highlight-foreground(the editing card-header bar and the selection/utility-menu triggers), and hover/highlight teal deepens slightly via the--boxel-dark-tealvalue change.Docs
gts-component-conventionsskill capturing these guidelines.<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