Skip to content

fix(replays): Shrink timeline hover timestamp#116268

Merged
scttcper merged 3 commits into
masterfrom
scttcper/replay-tooltip-size
May 27, 2026
Merged

fix(replays): Shrink timeline hover timestamp#116268
scttcper merged 3 commits into
masterfrom
scttcper/replay-tooltip-size

Conversation

@scttcper
Copy link
Copy Markdown
Member

@scttcper scttcper commented May 27, 2026

The replay scrubber hover timestamp was tall enough to trigger page overflow in the details layout.

before

image

after

image

The scrubber hover timestamp was tall enough to trigger page overflow in the replay details layout.

Use the smaller text size and tighter padding so the overlay stays inside the available space. Also keep the 0ms hover state visible instead of treating it as hidden.

Co-Authored-By: Codex GPT-5 <codex@openai.com>
@github-actions github-actions Bot added the Scope: Frontend Automatically applied to PRs that change frontend components label May 27, 2026
Keep the normal timestamp text size and reduce the hover label padding instead.

This keeps the label readable while still avoiding the replay page overflow.

Co-Authored-By: Codex GPT-5 <codex@openai.com>
Keep the tighter vertical padding but give the replay hover timestamp a little more room horizontally.

Co-Authored-By: Codex GPT-5 <codex@openai.com>
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 27, 2026

📊 Type Coverage Diff

✅ No new type safety issues introduced. Coverage: 93.58%

@scttcper scttcper marked this pull request as ready for review May 27, 2026 00:07
@scttcper scttcper requested a review from a team as a code owner May 27, 2026 00:07
@scttcper scttcper merged commit b93e3af into master May 27, 2026
75 checks passed
@scttcper scttcper deleted the scttcper/replay-tooltip-size branch May 27, 2026 15:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants