Skip to content

Conversation

eps1lon
Copy link
Collaborator

@eps1lon eps1lon commented Sep 10, 2025

We now show boundaries as slightly elevated by using a CSS box-shadow. This should help visualize hierarchy.

Had to switch to using <div> instead of <svg> since the box shadow filter in CSS would scale with the viewbox. For the stroke we used non-scaling-stroke to get a 1px outline but a similar trick doesn't seem available for SVG filter.
The scaling is still the same (i.e. we stretch the bbox of the roots to the available width of the Suspense tab and preserve aspect ratio of the bbox of the roots)

The dark mode variant may be a bit too subtle. Though it doesn't need to be apparent that there is a box shadow. The UI just needs to feel like there's elevation.

CleanShot.2025-09-12.at.10.25.39.mp4

@meta-cla meta-cla bot added the CLA Signed label Sep 10, 2025
@github-actions github-actions bot added the React Core Team Opened by a member of the React Core Team label Sep 10, 2025
@eps1lon eps1lon marked this pull request as ready for review September 10, 2025 10:31
@eps1lon eps1lon force-pushed the sebbie/09-10-_devtools_elevate_suspense_rects_to_visualize_hierarchy branch from e9074d6 to 958cc57 Compare September 10, 2025 13:08
@eps1lon eps1lon force-pushed the sebbie/09-10-_devtools_elevate_suspense_rects_to_visualize_hierarchy branch 2 times, most recently from c3c237f to 71928dd Compare September 17, 2025 13:51
@eps1lon eps1lon force-pushed the sebbie/09-10-_devtools_elevate_suspense_rects_to_visualize_hierarchy branch from 71928dd to a10e706 Compare September 18, 2025 16:24
@eps1lon eps1lon merged commit 755ceba into facebook:main Sep 18, 2025
241 checks passed
@eps1lon eps1lon deleted the sebbie/09-10-_devtools_elevate_suspense_rects_to_visualize_hierarchy branch September 18, 2025 16:37
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