-
Notifications
You must be signed in to change notification settings - Fork 47.2k
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
Standalone DevTools Profiler should work with no connected clients #22127
Comments
The messaging on this splash page is pretty important. Maybe we could keep it, but somehow show it inside of the Components tab– and the Profiler tab could be functional in a read-only (import only) form. |
That sounds like a good idea, yeah. If the components tab is the one that shows first by default too, people wouldn't miss this information. |
Hey, @bvaughn I can take a look at this during this weekend. 👍 |
Hey @hristo-kanchev. Sure, that'd be fine! I'm going to take a pass at updating the description to match my thinking. |
Sure thing. Thanks! |
Hi @bvaughn, I was taking a look at this issue yesterday and didn't notice @hristo-kanchev asked to take it and forgot to ask myself, sorry for any inconvenience and would understand if my solution is not considered. I think an alternative solution would be to add a "Profiler" section in the greeting/instructions page, that contains a button/link that redirects to the DevTools with the "Profiler" tab opened by default. I think it requires less changes (if any) directly to the DevTools component and can be made by just changing some things in the splash page and in the devtools-core. I took the time to quickly change the HTML of the splash page and it would look like this (might need some rewording): |
Hey @gabrieltrompiz. That's an interesting alternative proposal. Would be happy to review a proof of concept. |
Hiya @gabrieltrompiz! If you need help or want me to take over feel free to ping me. ^^ |
Going to mark this task as owned by the three of us so it will at least be clearly taken. We can coordinate the details going forward. Really appreciate the help! |
Thanks @bvaughn and @hristo-kanchev, I just submitted the PR for further discussion of the changes! |
When the standalone DevTools are opened without a running React application, this is what's currently shown:
Being able to use the standalone app without any connected clients, just to inspect saved profiles, would be a nice to have though. We should move the greeting message / instructions into the Components tab (when there's no connected apps) and let the Profiler be used to import previous profiles.
Note that doing this will likely require a new prop being passed to the outer DevTools component.
The text was updated successfully, but these errors were encountered: