Skip to content

feat(replay): Add tooltips to playlist next/prev buttons#107098

Merged
billyvg merged 2 commits intomasterfrom
feat-replay-add-tooltips-playlist-buttons
Jan 27, 2026
Merged

feat(replay): Add tooltips to playlist next/prev buttons#107098
billyvg merged 2 commits intomasterfrom
feat-replay-add-tooltips-playlist-buttons

Conversation

@billyvg
Copy link
Member

@billyvg billyvg commented Jan 27, 2026

image

@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Jan 27, 2026
@billyvg billyvg marked this pull request as ready for review January 27, 2026 20:30
@billyvg billyvg requested a review from a team as a code owner January 27, 2026 20:30
@@ -89,45 +90,57 @@ export default function ReplayDetailsPageBreadcrumbs({readerResult}: Props) {
<Flex>
<Flex align="center" gap="sm">
<div>
Copy link
Member

Choose a reason for hiding this comment

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

nit: div needed?

tooltip has optional wrapper thing
Very LGAF

Copy link
Member Author

Choose a reason for hiding this comment

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

The div groups the two < > together w/o gap, vs sharing same gap of < > id

@billyvg billyvg merged commit c103505 into master Jan 27, 2026
54 checks passed
@billyvg billyvg deleted the feat-replay-add-tooltips-playlist-buttons branch January 27, 2026 21:20
@github-actions github-actions bot locked and limited conversation to collaborators Feb 12, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

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