Caleydo / ordino

Target discovery platform for exploring rankings of genes, disease models, and other entities. @JKU-ICG @datavisyn
http://caleydo.org/
Mozilla Public License 2.0
11 stars 2 forks source link

fix: 🐛 useEffect missing check before setState #411

Closed kirangadhave closed 2 years ago

kirangadhave commented 2 years ago

An useEffect is missing a conditional check before changing the state for commentPanelVisible. This caused multiple rerenders. Discovered this bug when testing trrack integration with Ordino. Added the conditional check and updated the dependency array for the useEffect

Closes list issues numbers here

Developer Checklist (Definition of Done)

Summary of changes

Screenshots