Skip to content
This repository has been archived by the owner on Dec 23, 2020. It is now read-only.

Zoom does not fit in some cases #20

Open
koraa opened this issue Nov 2, 2014 · 5 comments
Open

Zoom does not fit in some cases #20

koraa opened this issue Nov 2, 2014 · 5 comments

Comments

@koraa
Copy link

koraa commented Nov 2, 2014

When searching for some cities, the result yields no pins at all.
Try "Hannover" for instance.

When you zoom out you might see some.

@koraa
Copy link
Author

koraa commented Nov 2, 2014

This is likely caused by the zoom being set to a default level.

As the simplest approach to mitigate this we could use an adaptive zoom: Zoom out until we see at least 3 pins.

This requires some math to get the rectangular canvas right.

@koraa
Copy link
Author

koraa commented Nov 2, 2014

We should take into account, that we don't want to scroll out too far if there is only one or two stores in a city.
Use some weighting method.

@koraa
Copy link
Author

koraa commented Dec 11, 2014

Time needed: 7h (This will include first repairing the JS, which is broken. This will be done together with #19)

@koraa
Copy link
Author

koraa commented Jan 17, 2015

We also need to take into account the size of the object we're looking at/searching for.
I therefor propose a view that focuses the entire geometry in question +15%/-30% percent zoom to optimize for pins in sight.
When there is no size given, we should go with minimum zoom to see at least N pins.

@koraa
Copy link
Author

koraa commented Jan 17, 2015

Time needed: 10h

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant