Created
Jun 26, 20:18
Started
Jun 26, 21:22
Completed
Jun 26, 23:21
DevOps handoff
Type
Chore
Shape
ui-only
Worktree Slug
center-timeline-card-layout
Repositories
mcritchie-studio
Release Train
—
Branch
feat/center-timeline-card-layout
Acceptance Criteria
Expected Test Plan
Checks Run
Agent Context
Follow-up UI polish to the /tasks/:id consolidated Stage Timeline cards (app/views/tasks/_consolidated_timeline.html.erb), requested live by the operator after PR #247 (badge fix) merged. Three changes: (1) center-justify the from->to stage transition badge row (add justify-center; drop ml-auto on the in-progress/reconstructed chip so it rides the centered group); (2) each crew member becomes an avatar-OVER-name centered column (flex-col items-center text-center); (3) the senior review pair renders as two avatar-over-name columns side by side, centered as a group. Pure presentation, no logic. Component test in test/controllers/tasks_controller_test.rb pins data-test=timeline-transition.justify-center, timeline-crew-member.flex-col, and 2 columns for the reviewed pair. Branch off release (which now contains the merged #247 helper fix + its tests).
Stage Timeline
Who handled each stage, the time it took (measured), and the model / tokens / cost reported (best-effort) — plus who's on it right now. — means the agent didn't report that metric.
Conversation
QA review feedback, agent handoffs, and follow-up notes for this task.
Senior review → APPROVED (2/2). Reviewers: Shannon (UI) + Carl (test), both high confidence. All 65 tests green (341 assertions). Acceptance: all 4 met — transition row justify-center (ml-auto dropped from both in-progress + reconstructed chips), crew avatar-over-name (flex-col items-center text-center), review pair → grid grid-cols-2 two equal columns, single actor → justify-center flex. Component test pins STATIC ERB classes (not Alpine-managed — Nokogiri gotcha N/A) and exercises both pair (reviewed block) and single (assembled block) branches via deploy-crew seed. No findings.
Re-submitted after review: 3 more operator-requested tweaks landed on PR #248 AFTER it was moved to reviewed — removed the 'in progress' pill, duration now ticks live as M:SS / H:MM, and the in-progress card backdrop pulses. New commit 9580cf0. Full suite + rubocop green at d9f5fbb5. Please re-review before merge.
Ready for QA review — operator-approved the look. PR #248 (head d409e0e) bundles the full /tasks/:id timeline-card redesign: (1) centered stage badges, (2) crew avatar-over-name, (3) review pair in 50/50 columns, (4) in-progress pill -> pulsing green backdrop, (5) live duration ticker M:SS / H:MM, (6) in-progress cards show only the current stage in its active-verb form ('Assembling'). New reusable Task::STAGE_ACTIVE_LABELS + Task.active_stage_label. Full suite + rubocop green at 15d4120f; [unit]+[component] tiers recorded. Local preview: http://localhost:3004/tasks/preview-live-qa-card
Senior re-review -> APPROVED (2/2). Reviewers: Shannon (UI heavy) + Carl (model/tests light), both high confidence. Re-review after the pulse/ticking-duration commits: all 230 tests green; all 9 acceptance criteria met (incl. pill removed, live ticker, pulsing bg, hides arrow/next, active-stage verb form). Live ticker verified as a correct Alpine x-data factory; Nokogiri-static assertions respected. Non-blocking follow-ups: (1) this PR orphaned the components/elapsed_label_script include at _consolidated_timeline.html.erb:58 -> now injects an unused window.elapsedLabel global; drop line 58 (keep it on _board.html.erb:43); (2) stale header doc lines 19-21 contradict the new code; update in same pass; (3) nits: motion-safe:animate-pulse, and a percent sign inside an ERB comment body. -> reviewed.
Sealed-bid sizing
Edit →Alex (PM)
—
Avi (PO)
SMALL
Dev
SMALL
Actual
XL
We emailed a one-tap sign-in link to . It expires shortly and can only be used once.
No email? Check spam, or close this and try again.