GreenInfo-Network / nyc-crash-mapper

React Redux web application for viewing, filtering, & comparing trends of automobile collision data published by the NYPD.
http://www.crashmapper.org
MIT License
16 stars 4 forks source link

De-selecting area or changing area type: Don't zoom citywide #73

Closed gregallensworth closed 6 years ago

gregallensworth commented 6 years ago

Noted in https://github.com/GreenInfo-Network/nyc-crash-mapper-chart-view/issues/73#issuecomment-359574748 regarding Intersections, but a good usability move.

Say you want to pan to a neighboring area (easily seen by the sudden cutoff of crashes)

Such "visually jarring" behavior has tested poorly on other projects in the past. I would recommend disabling this behavior.

gregallensworth commented 6 years ago

This is now in place. DR has a meeting with client in a few days, to determine whether this new behavior is preferred. If they like it, we can go ahead and close this issue. If not, check the commit and see about undoing the change.

gregallensworth commented 6 years ago

Approved by client.