Skip to content
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

[DevTools] Add React Devtools Extension Startup and File Structure to OVERVIEW.md #22283

Merged
merged 3 commits into from
Sep 9, 2021

Conversation

lunaruan
Copy link
Contributor

@lunaruan lunaruan commented Sep 9, 2021

I made an excalidraw to map out how the React Devtools Extension files are laid out and how initialization works.

Future TODO: Expand this section, add how standalone/inline files are laid out, and make this not an image and/or add accessibility text

…view

I previously made an excalidraw to map out how the React Devtools Extension code is laid out. This should probably be expanded and have better accessibility (because it's an image and not text) but leaving this here so it's more easily accessible in the meantime.
@facebook-github-bot facebook-github-bot added CLA Signed React Core Team Opened by a member of the React Core Team labels Sep 9, 2021
@lunaruan lunaruan changed the title Added React Devtools Extension Startup and File Structure to the Over… [DevTools] Add React Devtools Extension Startup and File Structure to OVERVIEW.md Sep 9, 2021
Copy link
Contributor

@jstejada jstejada left a comment

Choose a reason for hiding this comment

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

yay! thank you!


## Package Specific Details

### Devtools Extension
Copy link
Contributor

Choose a reason for hiding this comment

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

are these the sections to fill out later?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Eep good callout removed the extra title

Copy link
Contributor

Choose a reason for hiding this comment

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

nit: I would maybe combine and reword these titles slightly

I'm not sure this is "package specific" so much as it is...a diagram of how the pieces fit together at a high level. Maybe "Overview diagram" or something?

Copy link
Contributor

@bvaughn bvaughn left a comment

Choose a reason for hiding this comment

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

Nice 😄


## Package Specific Details

### Devtools Extension
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: I would maybe combine and reword these titles slightly

I'm not sure this is "package specific" so much as it is...a diagram of how the pieces fit together at a high level. Maybe "Overview diagram" or something?

@lunaruan lunaruan merged commit cb8a506 into main Sep 9, 2021
zhengjitf pushed a commit to zhengjitf/react that referenced this pull request Apr 15, 2022
@kassens kassens deleted the lunaruan-patch-1 branch November 29, 2022 22:24
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.

None yet

4 participants