-
Notifications
You must be signed in to change notification settings - Fork 56
Bad performance with Destiny Item Manager site #195
Copy link
Copy link
Closed
Labels
Description
This morning I got a report of DevTools performing poorly on the Destiny Item Manager app (facebook/react-devtools#681 (comment))
I think this may be due to props being unexpectedly large (which we send through the bridge and try to render in the right hand side, inspected element panel). Initial Chrome profiling shows us spending a lot of time in this panel.
I think this is made worse because of how we auto-select on hover when using the inspect button, and select triggers us to fetch props- so if you really quickly hover over a lot of items, we ask for all of their props.
I think we may want to do two things:
- Re-think our serialization / hydration stuff so we can better handle huge props
- Don't send the inspect request on mouse-over for the inspect button mode, but wait until the user clicks and confirms
Reactions are currently unavailable