Auto-zoom behavior on small screens

civicmapper / flush-it

See how far and where wastewater travels to the Allegheny County Sanitary Authority (ALCOSAN) treatement facility.

http://flush-it.civicmapper.com

MIT License

1 stars 4 forks source link

Auto-zoom behavior on small screens #11

Closed gassc closed 5 years ago

gassc commented 5 years ago

User comment:

the smaller version zooms out to full view after mapping the route. this is a bit confusing because the route is not entirely visible in the small window until you zoom in

Will investigate; not sure if this is a bug or an expectation about behavior. Either way, possible solution would be to disable zoom-to-extents behavior on small screens, using standard screen breakpoints.

gassc commented 5 years ago

Looks to be a bug, with zoom-to-extents zooming to far in at what looks to be the centroid of the geometry extent of the trace result.

gassc commented 5 years ago

addressed with https://github.com/civicmapper/flush-it/commit/d9e545c7fea71764926fb5eec64f88890029fbb0