fix(web): hovering an expanded command row no longer shifts its text - #12614
atharvadeosthale wants to merge 1 commit into
Conversation
ApprovabilityVerdict: Approved at Macroscope's review found this PR approvable — This one-file UI fix reserves timestamp width only for expanded command rows, preventing hover-induced text reflow while leaving collapsed rows unchanged. The change is small, self-contained, and does not affect schemas, defaults, infrastructure, or sensitive code. You can add or adjust custom eligibility rules. Learn more. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository: pingdotgg/t3code/.coderabbit.yaml Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (1)
Included review availability: Your plan provides up to 10 included reviews per hour; 9 remain after this review. 📝 WalkthroughWalkthrough
ChangesWork entry timestamp positioning
Priority: ⬇️ Low Estimated code review effort: 1 (Trivial) | ~3 minutes Change: Bug fix Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
|
Hey @juliusmarminge, slight bump for this PR, this fixes a problem that irritates me everyday :D Sorry for the ping, I will avoid pinging for this PR from now on. |
What Changed
On an expanded work entry row, the hover timestamp now always reserves its width and stays invisible until the row is hovered or focused. Collapsed rows are unchanged: the timestamp still takes no space until hover.
One added
classNameprop inMessagesTimeline.tsx.Why
Since #8641, the timestamp switches from
absolutetostaticon hover so it never overlaps the row text. On a collapsed row that only truncates the one-line preview a little earlier. On an expanded row the command text wraps, so taking width away on hover re-wraps the whole command, and the block jumps on every hover in and hover out.Reserving the width up front on expanded rows means hover only changes opacity.
Checked in the dev web app against a copy of real thread data, on an expanded 510-character command row. Text width (575.39px), text height (136.5px), and row height (177.625px) are identical hovered and not hovered. Collapsed rows still compute
position: absolutefor the hidden timestamp.UI Changes
Before:
before.mp4
After:
after.mp4
Checklist
Claude Fable 5.1 via Claude Code in T3 Code
Summary by CodeRabbit