Skip to content

[DevTools] Layout for Suspense tab #34042

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
Jul 30, 2025
Merged

Conversation

eps1lon
Copy link
Collaborator

@eps1lon eps1lon commented Jul 29, 2025

Based on the Components tab UI but with an additional column on the left.

With the additional left column it got a bit more crowded so I'm using the same layout Chrome's Sources panel uses: Navigator (here: tree list) and content (here: rects) are always two columns. Debugger (here: inspected element) is the rightmost column if enough horizontal space is available. Otherwise it gets pushed below navigator/content.

Icons and borders aren't aligned yet. I'll adjust those once these panes actually get content. I just needed something that allows me to easily make room for the pane I'm currently focusing on. Might move the toggle icons to the rects pane since timeline pane has very little space to deal with.

CleanShot.2025-07-29.at.19.07.12.mp4

@meta-cla meta-cla bot added the CLA Signed label Jul 29, 2025
@github-actions github-actions bot added the React Core Team Opened by a member of the React Core Team label Jul 29, 2025
@@ -46,8 +52,10 @@ type Props = {
type: IconType,
};

const materialIconsViewBox = '0 -960 960 960';
Copy link
Collaborator Author

@eps1lon eps1lon Jul 29, 2025

Choose a reason for hiding this comment

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

It's just the default https://fonts.google.com/icons comes with. You can scale the paths to adhere to the default viewbox but I couldn't find a tool for that quickly and none of the LLMs produced a result I could use.
The Material icons are the ones Chrome is using for the sidebar toggles.

They're license under Apache 2: https://developers.google.com/fonts/docs/material_icons#licensing

@eps1lon eps1lon force-pushed the sebbie/suspensetabui branch from 4f56a25 to 2f33673 Compare July 29, 2025 17:35
element.releasePointerCapture(event.pointerId);
};

const onResizeTree = (event: SyntheticPointerEvent<HTMLElement>) => {
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

There's probably an abstraction here that can be shared with the Components tab. I had to slightly adjust where and what percentage values with set though so I opted for duplicating. At least during experimentation phase.

@eps1lon eps1lon requested review from sebmarkbage and hoxyq July 29, 2025 17:41
@eps1lon eps1lon marked this pull request as ready for review July 29, 2025 17:41
@eps1lon eps1lon merged commit 36c63d4 into facebook:main Jul 30, 2025
245 checks passed
@eps1lon eps1lon deleted the sebbie/suspensetabui branch July 30, 2025 05:12
github-actions bot pushed a commit to code/lib-react that referenced this pull request Aug 2, 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