colorado-cancer-center / ecco

An interactive resource for exploring cancer data in Colorado
https://coe-ecco.org
Apache License 2.0
1 stars 2 forks source link

Frontend touchups #52

Closed vincerubinetti closed 5 months ago

vincerubinetti commented 5 months ago

Still want to test this for leaflet memory leaks when resetting it.

netlify[bot] commented 5 months ago

Deploy Preview for exploring-cancer-in-colorado ready!

Name Link
Latest commit 5036a91c752d74989a2a655e9257dabd8bf3140a
Latest deploy log https://app.netlify.com/sites/exploring-cancer-in-colorado/deploys/65fdbaa5ca4d11000835fc93
Deploy Preview https://deploy-preview-52--exploring-cancer-in-colorado.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

vincerubinetti commented 5 months ago

I wonder if the "reset" button should also reset the chosen geography, measure, etc.? It's really a matter of opinion, but I thought I'd mention it.

I figured people maybe wouldn't want to change the data that they were looking at, and then the reset button would more be the aesthetical map stuff. However, Cydney requested this feature and I'm still not sure exactly what she's looking for, but she mentioned graphical bugs or something with the map, so hopefully this hard reset of the map should address that.