maproulette / maproulette3

MapRoulette, the micro-tasking tool for OpenStreetMap
https://maproulette.org
MIT License
128 stars 34 forks source link

When a Task contains a Point as well as a Linestring, make sure the Point lies within the viewport #451

Closed mvexel closed 6 years ago

mvexel commented 6 years ago

Consider task http://maproulette.org/mr3/challenge/367/task/367271

Upon initial load, depending on zoom level, the marker for the Point may lie outside the viewport, since the Linestring is long.

The Point likely indicates the location that needs attention.

So we should make sure the Point is always visible in the viewport as the task is loaded.

nrotstan commented 6 years ago

Clicking the "crosshairs" icon on the right side to fit the map to the task features does include the point on that task, as expected.

So I think the problem here is our preservation of the zoom level moving from task to task within a challenge: we should only preserve the zoom if all the features of the new task will be displayed on the map at that zoom level, otherwise we should automatically fit the map to the task features.