Skip to content

Conversation

@Jesse-Box
Copy link
Contributor

@Jesse-Box Jesse-Box commented Nov 28, 2025

Problem

Both in the Replay Details and Span Waterfall, we have this ability to paginate to sibling events. However, the implementations are inconsistent.

Replay Details

CleanShot 2025-11-28 at 14 04 39@2x

Span Waterfall

CleanShot 2025-11-28 at 14 06 48@2x

Goal

This PR isn't about implementing the pagination UI consistently; rather, it's about modifying the breadcrumbs UI so that we can reach that end goal. Specifically, this PR swaps out the chevron icon for a forward slash (new icon).

Before

CleanShot 2025-11-28 at 13 58 09@2x

After

CleanShot 2025-11-28 at 13 58 37@2x

Solution

We're making this change because we want to reach an end goal where we have this

Ideal end state

CleanShot 2025-11-28 at 14 26 50@2x

End state without this change

CleanShot 2025-11-28 at 14 27 33@2x

Solution Explanation

In the realm of computing, the slash is frequently used to indicate a file path.

@Jesse-Box Jesse-Box requested a review from a team as a code owner November 28, 2025 13:01
@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Nov 28, 2025
@Jesse-Box Jesse-Box merged commit 9c1612a into master Nov 28, 2025
48 checks passed
@Jesse-Box Jesse-Box deleted the feat/icon/slashForward branch November 28, 2025 14:51
roaga pushed a commit that referenced this pull request Dec 2, 2025
## Problem
Both in the Replay Details and Span Waterfall, we have this ability to
paginate to sibling events. However, the implementations are
inconsistent.

### Replay Details
<img width="2880" height="2042" alt="CleanShot 2025-11-28 at 14 04
39@2x"
src="https://github.com/user-attachments/assets/ff6a81ce-dda5-4db6-b775-a3183a957eaf"
/>

### Span Waterfall
<img width="2880" height="2042" alt="CleanShot 2025-11-28 at 14 06
48@2x"
src="https://github.com/user-attachments/assets/5fda7778-712d-4dfe-aa9b-e2c6a4f5fe2b"
/>

## Goal
This PR isn't about implementing the pagination UI consistently; rather,
it's about modifying the breadcrumbs UI so that we can reach that end
goal. Specifically, this PR swaps out the chevron icon for a forward
slash (new icon).

### Before
<img width="2352" height="277" alt="CleanShot 2025-11-28 at 13 58 09@2x"
src="https://github.com/user-attachments/assets/8e4c6638-7785-4060-b0a4-1ba978589989"
/>

### After
<img width="2351" height="278" alt="CleanShot 2025-11-28 at 13 58 37@2x"
src="https://github.com/user-attachments/assets/9ada7a5f-13a9-48dd-824e-e6fdb2fc01a1"
/>

## Solution
We're making this change because we want to reach an end goal where we
have this

### Ideal end state
<img width="1978" height="284" alt="CleanShot 2025-11-28 at 14 26 50@2x"
src="https://github.com/user-attachments/assets/b3026ca0-3b8d-4898-bf76-933cf74e80b6"
/>

### End state without this change
<img width="1976" height="286" alt="CleanShot 2025-11-28 at 14 27 33@2x"
src="https://github.com/user-attachments/assets/02d49a61-cae7-4db2-85ce-eb77cfedafbf"
/>

## Solution Explanation
In the realm of computing, the slash is frequently used to indicate a
file path.
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.

3 participants