open-source-labs / Reactime

Developer tool for time travel debugging and performance monitoring in React applications.
https://www.reacti.me
MIT License
2.17k stars 200 forks source link

Pre Launch Final Bug Squashing Pt 1 #208

Closed kevinfey closed 4 years ago

kevinfey commented 4 years ago

Types of changes

Purpose

Address black screen of death on History tab within React Router apps Fix styling error in AtomsRelationship.tsx Clean up code in StateRoutes Clean up code in PerfView

Approach

Change line 111 in History.tsx. Now avoids a crash caused by moving to a route with no children.

        if (d.data.stateSnapshot.children[0].componentData.actualDuration){
          loadTime = d.data.stateSnapshot.children[0].componentData.actualDuration;
        } else{
          loadTime = 1;
        }

Set height of Sankey chart to 98% to allow space for buttons and fix layout bug.

Learning

Testing a broader range of apps exposed new bugs in code.

haejinjo commented 4 years ago

This PR was closed because it was made to the wrong repository!