Skip to content

Conversation

@eps1lon
Copy link
Collaborator

@eps1lon eps1lon commented Sep 4, 2025

I noticed in internal apps that inspecting an element took a while on the first click. I used the new Suspense tab to find what was suspending the shell :taps-head:

Before:

CleanShot.2025-09-04.at.13.52.03.mp4

After:

CleanShot.2025-09-04.at.14.04.56.mp4

@meta-cla meta-cla bot added the CLA Signed label Sep 4, 2025
@github-actions github-actions bot added the React Core Team Opened by a member of the React Core Team label Sep 4, 2025
@eps1lon eps1lon requested a review from hoxyq September 4, 2025 12:11
@eps1lon eps1lon marked this pull request as ready for review September 4, 2025 12:11
@sebmarkbage
Copy link
Collaborator

I didn't notice this since I always run in the "Always Open Files in External Editor" mode which disables the button. 😏

Btw, I have a todo to actually just deprecate this button and eventually remove it. Since I feel like it's not worth existing separately from the <> button since it's the only thing that has a separate button where as other links don't have that.

However, as a teaching opportunity it's nice to teach people about the new option to always open in external first (which removes the button), or teach people to jump to source code and then use the "Open in Editor" button from there which is a pattern that always works.

@eps1lon eps1lon merged commit e2cc315 into facebook:main Sep 4, 2025
250 checks passed
@eps1lon eps1lon deleted the sebbie/09-04-_devtools_don_t_suspend_shell_while_retrieving_original_source_for_open-in-editor_ branch September 4, 2025 14:42
EugeneChoi4 pushed a commit to EugeneChoi4/react that referenced this pull request Sep 4, 2025
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