fix(webapp): choose an article by clicking its name (#1268) #1270

Merged
claude-bot merged 1 commit from fix/article-picker-row-click into main 2026-10-29 17:55:17 +00:00
Collaborator

Closes #1268.

  • ArticlePicker: each result is now one button whose text is the name, styled as plain text (underlined on hover, 24px target, 44px on a coarse pointer). The separate "Choose" link is gone. The accessible name stays Choose <title>, so the label-in-name check (2.5.3) holds and the existing tests still find it.
  • The picker's styles move from PrepSheet.css (scoped to .qb-prep) into a new ArticlePicker.css. The map's marker form had none of them, which is why it showed "AramoreChoose".
  • Changelog fragment 1268-article-picker-click-name.fixed.md.

Checked: vitest for src/pages/wiki/Map.test.jsx and src/pages/prep (186 passed); eslint with 0 errors; changelog.py check.
Not checked: no render-check. The journeys stack wasn't brought up in this session, so the look in the prep drawer and the map dialog has not been seen in a browser.

🤖 Generated with Claude Code

Closes #1268. - `ArticlePicker`: each result is now one button whose text is the name, styled as plain text (underlined on hover, 24px target, 44px on a coarse pointer). The separate "Choose" link is gone. The accessible name stays `Choose <title>`, so the label-in-name check (2.5.3) holds and the existing tests still find it. - The picker's styles move from `PrepSheet.css` (scoped to `.qb-prep`) into a new `ArticlePicker.css`. The map's marker form had none of them, which is why it showed "AramoreChoose". - Changelog fragment `1268-article-picker-click-name.fixed.md`. **Checked:** vitest for `src/pages/wiki/Map.test.jsx` and `src/pages/prep` (186 passed); eslint with 0 errors; `changelog.py check`. **Not checked:** no render-check. The journeys stack wasn't brought up in this session, so the look in the prep drawer and the map dialog has not been seen in a browser. 🤖 Generated with [Claude Code](https://claude.com/claude-code)
fix(webapp): choose an article by clicking its name (#1268)
All checks were successful
CI / Does this change touch the recording path? (pull_request) Successful in 11s
CI / Synthetic session harness (no GPU, no LLM) (pull_request) Has been skipped
CI / Which journeys does this change need? (pull_request) Successful in 13s
CI / Bot/backend version sync (pull_request) Successful in 27s
CI / Feedback site tests (pull_request) Successful in 48s
CI / Backend lint (ruff) (pull_request) Successful in 1m15s
CI / Summarisation accuracy eval harness (stub provider) (pull_request) Successful in 2m14s
CI / Bot tests and audit (pull_request) Successful in 3m34s
CI / Docker image build (pull_request) Successful in 4m36s
CI / Frontend (shard 2/2) (pull_request) Successful in 15m26s
CI / Frontend (shard 1/2) (pull_request) Successful in 19m14s
CI / End-to-end journeys (desktop) (pull_request) Successful in 27m53s
CI / Backend migration, tests, and audit (pull_request) Successful in 57m55s
64d7a5c9f5
The picker listed each result as a title with a "Choose" link beside it.
A result is now one button whose text is the name, styled as plain text,
with the accessible name "Choose <title>" kept. Its styles move out of
PrepSheet.css (scoped to .qb-prep) into ArticlePicker.css, so the map's
marker form, which had none of them, no longer shows "AramoreChoose".

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
claude-bot deleted branch fix/article-picker-row-click 2026-10-29 17:55:18 +00:00
Sign in to join this conversation.
No description provided.