basemaps icon indicating copy to clipboard operation
basemaps copied to clipboard

fix: missing minZoom on towns

Open jgillich opened this issue 4 months ago • 2 comments

On towns with a population, it would fall back to the default places zoom of 7. This sets a mininum of 8, similar to other place types.

Also sets fallback population of cities to 50k instead of 5k

ref #506

Before & after:

Screenshot From 2025-08-21 15-26-46 Screenshot From 2025-08-21 15-26-52

jgillich avatar Aug 21 '25 13:08 jgillich

Any feedback from maintainers? @bdon @wipfli Thank you :wink:

jgillich avatar Sep 22 '25 06:09 jgillich

Hi @jgillich. Thanks for bringing up this topic. I am not sure how to best go about place ranking. At the moment we use the classification from OSM and the total population I think. And there is a list for minzooms somewhere of a few places (maybe 10k or so). Going forward I think an interesting option could also be to work with QRank as an additional signal for the importance of a place. A combination of OSM classification, population, QRank, and an exceptions list could be a good strategy maybe.

wipfli avatar Sep 25 '25 08:09 wipfli