Skip to content

Commit abe98a1

Browse files
lukemeliaclaude
andcommitted
Sync the card-chooser selection ring with the tag, instant on hover (CS-11330)
The selected ring faded in over `box-shadow 0.2s` while the selection tag/chip render instantly, so the border appeared ~0.2s after the tag. Set `transition: none` on the selected ring so the two appear together. The `:hover` variant needs it too: a card is normally hovered at the moment it's clicked to select, so that rule governs the ring's first paint — without it the ring still animated in late on select-while-hovered. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
1 parent aed874d commit abe98a1

1 file changed

Lines changed: 8 additions & 2 deletions

File tree

packages/host/app/components/card-search/item-button.gts

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -448,9 +448,15 @@ export default class ItemButton extends Component<Signature> {
448448
which the portaled catalog item didn't reliably inherit — leaving
449449
only the thin 1px button border) so the ring stays a full-weight
450450
4px whether or not the item is hovered, darkening on hover to match
451-
the rest of the Adorn treatment. */
452-
.catalog-item.adorn.selected {
451+
the rest of the Adorn treatment. `transition: none` makes the ring
452+
appear in lockstep with the selection tag/chip (which render
453+
instantly) instead of fading in ~0.2s later — and it's needed on the
454+
hover variant too, since a card is normally hovered at the moment
455+
it's clicked to select, so that rule governs the ring's first paint. */
456+
.catalog-item.adorn.selected,
457+
.catalog-item.adorn.selected:hover {
453458
box-shadow: 0 0 0 0.25rem var(--boxel-highlight);
459+
transition: none;
454460
}
455461
.catalog-item.adorn.selected:hover {
456462
box-shadow: 0 0 0 0.25rem var(--boxel-highlight-hover);

0 commit comments

Comments
 (0)