MotirBuilding in public
MOTIR · moooon
onMotir
You’re viewing a public project. Anyone can view it — no account needed. Sign in to submit, upvote, or comment on requests.View-only — you can’t edit work items
MOTIR-1306

2.5 follow-up — /items rows open the quick-view peek on row click; remove the per-row eye trigger

Done
Description

On the /items page, make a PLAIN click on a work-item row open the shared quick-view peek, and REMOVE the per-row eye trigger — the eye becomes redundant once the row itself peeks (motir-core, frontend only).

Shipped reality

  • /items rows render in two views — the flat List (IssueListTable / issueColumns.tsx) and the Tree (IssueTreeStaticTable); the per-row action bar WorkItemRowActions.tsx renders <QuickViewTrigger> — the Eye button (Subtask 2.5.19) that opens ?peek=<id>.
  • <IssueQuickViewController /> is already mounted on /items, and RelationshipPeekLink/usePeekOpen is the established peek-on-click pattern.

The change

  • Make the row's PRIMARY navigable link (the identifier/title link) open the peek on a plain click — reuse the RelationshipPeekLink behavior (usePeekOpen + an anchor that keeps the real /items/<id> href so ⌘/ctrl/middle-click opens the full page in a new tab). Apply to BOTH the List and Tree row renderers.
  • Remove QuickViewTrigger (the eye) from WorkItemRowActions (and delete the component if it has no other caller — it does not, per grep). Keep the other row actions intact.
  • The full detail page stays reachable via modifier/middle-click AND the peek panel's own "open full item" affordance (shipped in IssueQuickViewPanel).

Acceptance criteria

  • Plain-clicking a row (List and Tree) opens the quick-view peek; the per-row eye is gone; ⌘/ctrl/middle-click still opens the full page in a new tab; inline-edit / row controls still work (the peek click must not hijack clicks on interactive cells).
  • QuickViewTrigger removed (and deleted if unreferenced); no dangling import; tokens via --el-*/shape only.
  • Update the issue-list E2E: row click opens the peek (no eye), a modified click opens the detail page; wait on the ?peek URL/panel signal.

Context refs

  • Reuses usePeekOpen/RelationshipPeekLink/IssueQuickView (shipped). Files: WorkItemRowActions.tsx, QuickViewTrigger.tsx, issueColumns.tsx, IssueListTable.tsx, IssueTreeStaticTable.tsx. Interaction/token rules per motir-core/CLAUDE.md.
Status
Done
Type
Sub-task