Skip to content

fix(web): emphasise only the chosen approach once one is chosen - #28

Merged
TheGreatAxios merged 2 commits into
mainfrom
cl-11869-builder-keeps-recommended-option-highlighted-after-choosing
Oct 31, 2026
Merged

TheGreatAxios merged 2 commits into
mainfrom
cl-11869-builder-keeps-recommended-option-highlighted-after-choosing

Conversation

@TheGreatAxios

@TheGreatAxios TheGreatAxios commented Oct 31, 2026 •

Copy link
Copy Markdown
Contributor

Closes CL-11869

What changed

  • Approach cards mark state with data-recommended and data-chosen (set on the chosen card, and on the clicked card while the choice is sending).
  • CSS emphasises the chosen card, and the recommended card only while no card in the grid is chosen. The recommended card keeps its Recommended badge.

Verification

  • bun run typecheck, bun run lint, bun run format:check pass.
  • No other component in apps/web/src has a recommended-vs-chosen pattern.

The recommended approach card kept its emphasis after the person chose
another, so it still looked selected. Emphasis now follows the choice;
the recommended card keeps its badge.

Closes CL-11869
@linear-code

linear-code Bot commented Oct 31, 2026

Copy link
Copy Markdown

CL-11869

Cards carry data-recommended and data-chosen; the clicked card counts as
chosen while the choice is sending.
@TheGreatAxios
TheGreatAxios marked this pull request as ready for review October 31, 2026 17:03
@corbits-triage

corbits-triage Bot commented Oct 31, 2026 •

Copy link
Copy Markdown

Triage: Pass

No changes needed from you. A maintainer will review this pull request.

Triage will approve, request review.

@TheGreatAxios
TheGreatAxios added this pull request to the merge queue Oct 31, 2026
Merged via the queue into main with commit 4779d4b Oct 31, 2026
8 checks passed
@TheGreatAxios

TheGreatAxios commented Oct 31, 2026 •

Copy link
Copy Markdown
Contributor Author

Before and after

Real ApproachesComparison component, app stylesheets, light theme. Row 3 is the bug case: before, the recommended card stays emphasised; after, only the chosen card is.

BeforeAfter

Source: https://gist.github.com/TheGreatAxios/9bdfb4b7c07c66a721d98bc4023ddfcc

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant