Closed bliitzkrieg closed 3 years ago
@chopchop505 Should be good for review. Do you want to use a different icon for the toggle?
I noticed one more thing that should probably be adjusted...
Right now there's some default styling for the incident map component that's mixed in the incident analysis page's css (.incident-map-container
and .incident-map-concurrent-layer
in incident-analysis.scss
), but I think that probably belongs in a file specifically for the incident map component, so that it's reusable. Otherwise you'd have to set the styling for that stuff again for any page where you want to reuse the concurrent incidents toggle button.
Should be good for review
Looks great! I'll merge it in.
Overview
Added ability to toggle concurrent incidents on map inside Incident analysis view
GitHub Issues
481
Changes
Screenshots / Videos
Steps to Test