Skip to content

Conversation

hoxyq
Copy link
Contributor

@hoxyq hoxyq commented Jun 9, 2025

Summary

Minor changes around css and styling of Settings dialog.

  1. :root selector was updated to :is(:root, :host) to make css variables available on Shadow Root
  2. CSS tweaks around Settings dialog: removed references to deleted styles, removed unused styles, ironed out styling for cases when input styles are enhanced by user agent stylesheet

How did you test this change?

Before After
Screenshot 2025-06-09 at 15 35 55 Screenshot 2025-06-09 at 15 26 12
Screenshot 2025-06-09 at 15 36 02 Screenshot 2025-06-09 at 15 26 25
Screenshot 2025-06-09 at 15 36 05 Screenshot 2025-06-09 at 15 26 30
Screenshot 2025-06-09 at 15 36 12 Screenshot 2025-06-09 at 15 26 37

Copy link

@Hardanish-Singh Hardanish-Singh left a comment

Choose a reason for hiding this comment

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

LGTM!

@hoxyq hoxyq force-pushed the react-devtools/support-shadow-dom branch from cd8710f to 92d92b1 Compare June 9, 2025 15:50
@hoxyq hoxyq merged commit 80c03eb into facebook:main Jun 9, 2025
241 checks passed
@hoxyq hoxyq deleted the react-devtools/support-shadow-dom branch June 9, 2025 17:25
hoxyq added a commit that referenced this pull request Jun 27, 2025
Full list of changes:

* devtools: emit performance entries only when profiling
([hoxyq](https://github.com/hoxyq) in
[#33652](#33652))
* Get Server Component Function Location for Parent Stacks using Child's
Owner Stack ([sebmarkbage](https://github.com/sebmarkbage) in
[#33629](#33629))
* Added minimum indent size to Component Tree
([jsdf](https://github.com/jsdf) in
[#33517](#33517))
* [devtools-shell] layout options for testing
([jsdf](https://github.com/jsdf) in
[#33516](#33516))
* Remove feature flag enableRenderableContext
([kassens](https://github.com/kassens) in
[#33505](#33505))
* refactor[devtools]: update css for settings and support css variables
in shadow dom scnenario ([hoxyq](https://github.com/hoxyq) in
[#33487](#33487))
* [mcp] Add MCP tool to print out the component tree of the currently
open React App ([jorge-cab](https://github.com/jorge-cab) in
[#33305](#33305))
* [scripts] Switch back to flow parser for prettier
([rickhanlonii](https://github.com/rickhanlonii) in
[#33414](#33414))
* upgrade json5 ([rickhanlonii](https://github.com/rickhanlonii) in
[#33358](#33358))
* Get source location from structured callsites in prepareStackTrace
([sebmarkbage](https://github.com/sebmarkbage) in
[#33143](#33143))
* Clean up enableSiblingPrerendering flag
([jackpope](https://github.com/jackpope) in
[#32319](#32319))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants