Issue 32: add ISO8601 timestamp to date display title attribute#894
Conversation
|
Note Reviews pausedIt looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the Use the following commands to manage reviews:
Use the checkboxes below for quick actions:
WalkthroughReplaced the long, locale-specific tooltip content with an ISO8601 string: Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes 🚥 Pre-merge checks | ✅ 4✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches
🧪 Generate unit tests (beta)
No actionable comments were generated in the recent review. 🎉 🧹 Recent nitpick comments
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
4e18385 to
8f265a9
Compare
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Fix all issues with AI agents
In `@packages/web/src/app/`[domain]/components/DisplayDate.tsx:
- Line 29: Remove the native title attribute on the <span> (currently using
iso8601Date) to avoid the browser tooltip overlapping the Radix <Tooltip>;
instead, surface the ISO8601 string inside the existing TooltipContent (e.g. add
a secondary line or smaller text showing iso8601Date) within the DisplayDate
component so the TooltipTrigger/TooltipContent pair shows both the formatted
date and the raw ISO timestamp in one styled tooltip; update references to
iso8601Date, TooltipTrigger, TooltipContent and the span with className to
reflect the removal of title and the new content placement.
f7ef627 to
cc1bce6
Compare
Add ISO8601 (UTC) formatted timestamp as a native browser tooltip on all date displays. This allows users to easily copy the exact timestamp while still seeing human-readable relative times and formatted dates in the custom tooltip. Fixes sourcebot-dev#32
cc1bce6 to
d9e874b
Compare
Problem
#32
Solution
Add ISO8601 (UTC) formatted timestamp as a browser tooltip on all date displays (NOT only Last Indexed).
UI Verification
Before
After
Summary by CodeRabbit