tilery / pianoforte

Generic purpose map rendering in two flavours.
http://tiles.quaidorsay.fr/
Creative Commons Zero v1.0 Universal
28 stars 8 forks source link

Cities order #45

Closed kent1D closed 6 years ago

kent1D commented 6 years ago

Change strategy to order cities

First, we order them with capital field (an admin_level 4 capital should be shown before bigger cities)

Then we use city / town tag to order them

Then we use the population tag.

Should FIX #36

Additionally: Use shield-min-padding to avoid cutted letters

kent1D commented 6 years ago

Villeurbanne shouldn't be shown in place of Lyon:

villeurbanne_vs_lyon

Lorient versus Lanester:

lorient_vs_lanester

Around Paris, Bobigny, Créteil, Cergy, Versailles, Evry, Nanterre et Melun should have priority against Saint-Denis, Montreuil, Boulogne-Billancourt and Argenteuil, even if they are smaller:

ile_de_france
yohanboniface commented 6 years ago

Villeurbanne shouldn't be shown in place of Lyon:

have you investigated why? Seems to me that Lyon has priority over Villeurbanne on every order column, even without this PR. Sounds like a bug (maybe the missing NULLS LAST ?).