Events list timing (2/2): relative "time ago"#2636
Draft
mitul-s wants to merge 1 commit into
Draft
Conversation
Replace the bare wall-clock time on each row of the sidebar Events list with an auto-updating relative time (e.g. "2 minutes ago"), and move the exact date/time into the existing TimestampTooltip hover card so it is still available on demand. Signed-off-by: Cursor Agent <cursoragent@cursor.com>
Contributor
🦋 Changeset detectedLatest commit: 4f1ee6d The changes in this PR will be included in the next version bump. This PR includes changesets to release 16 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
The sidebar Events list currently shows each event's absolute wall-clock time (e.g.
3:47:18 PM). With no date, that's ambiguous for runs that span midnight or sleep for a day, and it doesn't convey recency at a glance.This PR (the relative-time approach, "Idea C") replaces the per-row clock time with an auto-updating relative label, keeping the exact timestamp on hover:
useShortTimeAgohook for the label ("Just now" / "N minutes ago" / "N hours ago" / "N days ago") and wraps it in the existingTimestampTooltip, whose hover card shows the detailed "x ago" plus full UTC and local date/time — so the exact instant is never lost and the cross-day ambiguity is resolved.Known trade-off versus the sibling PR: tightly-spaced events can all read "2 minutes ago," so this view favors "is it recent / is it stuck?" over "how long did each phase take?".
Scope is limited to the sidebar
EventsList(the component in the screenshot); the full Events tab table is unchanged.How did you test your changes?
pnpm --filter @workflow/web-shared typecheck— passes.pnpm --filter @workflow/web-shared build(incl. workspace deps via turbo) — passes.pnpm --filter @workflow/web-shared test— all pass except the 2 pre-existing failures intest/zstd-decoder.test.ts(zlib.zstdCompressSync is not a function), which fail identically on a cleanmainin this Node 22.14 environment and are unrelated to this change.biome checkon the changed component — no diagnostics.useShortTimeAgohook andTimestampTooltipand adds no new logic. Visual confirmation in a preview/dev run is recommended (the list renders when selecting a span in the trace viewer sidebar) — not performed in this environment.PR Checklist - Required to merge
pnpm changesetwas run to create a changelog for this PR (@workflow/web-sharedpatch)git commit --signoffon your commits)@vercel/workflowin a comment once the PR is ready, and the above checklist is complete