Skip to content

Issue 32: add ISO8601 timestamp to date display title attribute#894

Open
WilliamZhang9 wants to merge 1 commit intosourcebot-dev:mainfrom
WilliamZhang9:u/williamzhang/issue_32_last_indexed_time_format
Open

Issue 32: add ISO8601 timestamp to date display title attribute#894
WilliamZhang9 wants to merge 1 commit intosourcebot-dev:mainfrom
WilliamZhang9:u/williamzhang/issue_32_last_indexed_time_format

Conversation

@WilliamZhang9
Copy link

@WilliamZhang9 WilliamZhang9 commented Feb 14, 2026

Problem

#32

Solution

Add ISO8601 (UTC) formatted timestamp as a browser tooltip on all date displays (NOT only Last Indexed).

UI Verification

Before

image

After

image

Summary by CodeRabbit

  • Improvements
    • Tooltip now displays exact ISO 8601 UTC timestamps for dates, giving precise machine-readable date-time details.
    • Visible on-screen date formatting and tooltip behavior remain unchanged to preserve the familiar display.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 14, 2026

Note

Reviews paused

It 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 reviews.auto_review.auto_pause_after_reviewed_commits setting.

Use the following commands to manage reviews:

  • @coderabbitai resume to resume automatic reviews.
  • @coderabbitai review to trigger a single review.

Use the checkboxes below for quick actions:

  • ▶️ Resume reviews
  • 🔍 Trigger review

Walkthrough

Replaced the long, locale-specific tooltip content with an ISO8601 string: iso8601Date = date.toISOString() is now used for tooltip/title; visible rendering still uses getFormattedDate(date). Removed previous formatFullDate usage. No exported/public API changes.

Changes

Cohort / File(s) Summary
Date Display
packages/web/src/app/[domain]/components/DisplayDate.tsx
Removed formatFullDate usage, added iso8601Date = date.toISOString() and used it for TooltipContent/element title; visible content remains getFormattedDate(date). No exported/public declaration changes.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 4
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and specifically describes the main change: adding ISO8601 timestamp formatting to the date display's title attribute, directly addressing Issue 32.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Merge Conflict Detection ✅ Passed ✅ No merge conflicts detected when merging into main

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

No actionable comments were generated in the recent review. 🎉

🧹 Recent nitpick comments
packages/web/src/app/[domain]/components/DisplayDate.tsx (1)

21-23: Consider showing both human-readable and ISO8601 dates in the tooltip.

The raw ISO string (2026-02-15T00:52:38.881Z) is useful for copying/debugging but harder to read at a glance. Adding a human-readable line above it (as was there before) would give users the best of both worlds.

💡 Suggested enhancement
                 <TooltipContent>
-                    <p>{iso8601Date}</p>
+                    <p>{date.toLocaleString("en-US", { dateStyle: "long", timeStyle: "medium", timeZone: "UTC" })} UTC</p>
+                    <p className="text-xs text-muted-foreground">{iso8601Date}</p>
                 </TooltipContent>

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@WilliamZhang9 WilliamZhang9 force-pushed the u/williamzhang/issue_32_last_indexed_time_format branch from 4e18385 to 8f265a9 Compare February 14, 2026 19:31
@WilliamZhang9 WilliamZhang9 marked this pull request as ready for review February 14, 2026 23:11
Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

@WilliamZhang9 WilliamZhang9 force-pushed the u/williamzhang/issue_32_last_indexed_time_format branch 5 times, most recently from f7ef627 to cc1bce6 Compare February 15, 2026 00:23
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
@WilliamZhang9 WilliamZhang9 force-pushed the u/williamzhang/issue_32_last_indexed_time_format branch from cc1bce6 to d9e874b Compare February 15, 2026 01:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant