-
-
Notifications
You must be signed in to change notification settings - Fork 18
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
Add ability to "refresh" the devtools panel #169
Comments
Like the idea. As far as where to put it in the UI, I guess it depends on what the refresh is limited to. Are we just refreshing the components or will the refresh also clear any warnings and events in those tabs as well? |
Interesting point 🤔 , we could make it so that "refresh" applies only to the current component's data, we would have to rework some of the messaging, guess it would be related to #10 |
Yeah I guess really we could have two levels of refresh. One that triggers the refresh/re-walking of all components and one that was related to an individual component. By no means a suggestion for final UI but here's what I think a user would expect from the following placements for a button:
|
Oof the UX of having 3 refresh buttons 👀👀👀 Opened up react devtools, they don't even have a refresh button haha, the benefits of first party devtools support I guess. Vue.js devtools have refresh at the top-level (like you've got in the tab bar). I think that might the most reasonable approach to be fair: top-level "refresh" in the tab bar (what's been detailed as 1.), to allow the user to restart the panel without closing devtools or refreshing the page. It'll be an ok workaround if someone's in a pinch but hopefully they'll come report the issue here. What we could do for the component data refresh is do it under the hood when a "component" gets selected in the left pane (that would be good for an eventual move to more granular data loading: first load components/names, when a component gets selected, load the relevant data). |
Ha! ALL THE REFRESH BUTTONS! 😆
I agree. It's simple and clear for users I think.
Yeah really like that, makes a lot of sense. |
I'll split that more granular data loading as a separate issue, it'll be mostly a perf thing with UX benefits. |
Some issues wouldn't be as bad (#165 #151 ) if we had a way for the user to manually "refresh" (really re-initialise) the devtools panel
We can use the heroicons reload icon:
Some suggestions where we could add it (in the tabs, or next to the github icon), there's probably a way to add it in the "components" tab body cc @stephenoldham
The text was updated successfully, but these errors were encountered: