Skip to content

Conversation

sebmarkbage
Copy link
Collaborator

When you double click it will hide or show by jumping to the selected index or one step before the selected.

Let's you go from a suspense boundary into the timeline to find its position. I also highlight the step in the timeline when you hover the rect.

This only works if it's in the selected root but all of those should be merged into one single timeline.

One thing that's weird about the SuspenseNodes now is that they sometimes gets deleted but not always when they're resupended. Nested ones maybe? This means that if you double click to hide it, you can't double click again to show it. This seems like an unrelated bug that we should fix.

We could potentially repurpose the existing "Suspend" button in the toolbar to do this too, or maybe add another icon there.

@sebmarkbage sebmarkbage requested a review from eps1lon September 29, 2025 02:56
@meta-cla meta-cla bot added the CLA Signed label Sep 29, 2025
@github-actions github-actions bot added the React Core Team Opened by a member of the React Core Team label Sep 29, 2025
@sebmarkbage sebmarkbage merged commit d15d7fd into facebook:main Sep 29, 2025
247 checks passed
EugeneChoi4 pushed a commit to EugeneChoi4/react that referenced this pull request Sep 29, 2025
…e timeline (facebook#34642)

When you double click it will hide or show by jumping to the selected
index or one step before the selected.

Let's you go from a suspense boundary into the timeline to find its
position. I also highlight the step in the timeline when you hover the
rect.

This only works if it's in the selected root but all of those should be
merged into one single timeline.

One thing that's weird about the SuspenseNodes now is that they
sometimes gets deleted but not always when they're resupended. Nested
ones maybe? This means that if you double click to hide it, you can't
double click again to show it. This seems like an unrelated bug that we
should fix.

We could potentially repurpose the existing "Suspend" button in the
toolbar to do this too, or maybe add another icon there.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed React Core Team Opened by a member of the React Core Team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants