fix(web): keep thread actions in place on hover - #14974
RakshithBhat03 wants to merge 1 commit into
Conversation
ApprovabilityVerdict: Approved at Macroscope's review found this PR approvable — This is a focused one-file UI correction that reorders an existing timestamp so hover/focus visibility no longer shifts thread-action click targets. It introduces no API, configuration, security, billing, or deployment changes. No code changes detected at You can add or adjust custom eligibility rules. Learn more. |
39fa4fa to
8127f1a
Compare
|
Navigate logical layers of code changes, visualize relationships, and explore their blast radius. No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configuration
📒 Files selected for processing (1)
Included review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 9 remain after this review. 📝 WalkthroughWalkthroughThe work-entry timestamp moved to the start of the trailing controls, before thread actions and failure indicators. Its placement comment now describes this order. ChangesWork-entry timestamp order
Priority: ⬇️ Low Estimated code review effort: 2 (Simple) | ~5 minutes Change: Bug fix Suggested reviewers: Merge Risk: ⚪ Minimal · up to The timestamp now appears before the trailing controls, keeping them in place as it becomes visible. No actionable merge-blocking risk is established. 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
Full details: Description checkExplanation The description explains the problem, the layout change, and the verification performed. It includes before-and-after screenshots, but it does not provide the required scope and approval details or explain why this focused fix qualifies without prior approval.
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
|
superseded by #15281 |
Hovering a thread creation or subagent completion row reveals its timestamp and shifts the "Open chat" or "Open subagent" button under the pointer. The timestamp now appears before the trailing actions, keeping the buttons and disclosure chevron in place on hover and keyboard focus. This is a small layout correction to existing controls.
Tests: 213 focused timeline tests, web typecheck, targeted lint with warnings in unchanged code, and a real-client before/after hover check using a delegated subagent.
Before:
subagent-hover-before.mp4
After:
subagent-hover-after.mp4
Built by GPT 6.1 Sol in T3 Code through the Codex harness.