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-996

2.4.13 Remove the redundant neutral status Pill from the detail-page eyebrow

Done
Description

Estimate: 15m · Type: code

Remove the redundant status tag in the detail-page header eyebrow<Pill tone="neutral">{item.status}</Pill> at app/(authed)/issues/[key]/page.tsx:246. It is a 2.4.1 header-shell leftover: a static, read-only, untinted duplicate of the status already shown correctly elsewhere.

  • Canonical status lives in the rail: CoreFieldsPanel's editable StatusPicker renders status with the proper workflow-category tint (STATUS_TONE) and commits via changeStatusAction. The design specifies Status in the core-fields rail (design/work-items/design-notes.md L804) with category tones (L82) — NOT in the eyebrow (the detail.png eyebrow is [type] PROD-N · ancestor breadcrumb).
  • The eyebrow pill is tone="neutral" (grey, ignoring the mandated category colour) and read-only → off-design + redundant.

Do: delete the <Pill>…</Pill> from the eyebrow; remove the now-unused Pill import from page.tsx if nothing else in the file uses it. Keep the identifier, breadcrumb, and the right-hand cluster intact.

⚠️ Preserve the overflow fix. The breadcrumb currently shares a min-w-0 flex-1 cell WITH this Pill (the bug-issue-detail-eyebrow-overflows-viewport fix, page.tsx:233-249). When removing the Pill, keep the breadcrumb wrapped in its bounded min-w-0 flex-1 track so its inner truncate still has a bounded parent — otherwise a long ancestor chain reintroduces the viewport overflow. Update the stale comment to drop the "+ status Pill share a cell" wording.

Acceptance criteria

  • The detail-page eyebrow no longer renders the neutral status Pill; status remains shown + editable in the core-fields rail (unchanged).
  • A deep-ancestor item's eyebrow still truncates without overflowing the viewport (bounded min-w-0 flex-1 cell preserved).
  • Any test/snapshot asserting the eyebrow status pill is updated/removed; typecheck, lint, and the 2.4.6 detail E2E stay green.

Context refs

  • app/(authed)/issues/[key]/page.tsx:230-249 (eyebrow header — the Pill to remove + the overflow-fix cell + comment)
  • app/(authed)/issues/[key]/_components/CoreFieldsPanel.tsx (the canonical editable StatusPicker — unchanged)
  • design/work-items/design-notes.md L82 + L804 (status belongs in the rail, category-tinted)
Status
Done
Type
Sub-task