Move leisure=track and attraction=water_slide #5046
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixes #4726
Changes proposed in this pull request:
As discussed in #5041:
Move
leisure=track
fromamenity-line
(late in layer stack) to new layer betweentunnels
andlanduse-overlay
.Optimisation of layer ordering from end of road fill to roller coaster.
Move
attraction=water_slide
fromamenity-line
to new layer betweenaeroways
and roller coaster.[Also tweak to
highway=ford
SQL query to simplify hstore query]Test rendering with links to the example places:
Link:
Before
After
Ford as linear way
attraction=water_slide
Notes:
The git diff for
project.mml
is not easy to untangle. Here is the debug output fromnik4.py
showing the layer ordering (new layers highlighted):layers=landcover-low-zoom,landcover,landcover-line,water-lines-low-zoom,water-lines,water-areas,ocean-lz,ocean,landcover-area-symbols,marinas-area,water-barriers-line,water-barriers-poly,piers-poly,piers-line,water-barriers-point,bridge,buildings,tunnels,leisure-track,landuse-overlay,barriers,cliffs,ferry-routes,turning-circle-casing,highway-area-casing,roads-casing,highway-area-fill,roads-fill,turning-circle-fill,roads-low-zoom,guideways,bridges,aeroways,waterway-bridges,waterslide,roller-coaster-gap-fill,roller-coaster,aerialways,entrances,golf-line,necountries,admin-low-zoom,admin-mid-zoom,admin-high-zoom,power-minorline,power-line,tourism-boundary,protected-areas,trees,country-names,capital-names,state-names,placenames-medium,placenames-small,stations,junctions,bridge-text,county-names,amenity-points,amenity-line,power-towers,roads-text-ref-low-zoom,roads-text-ref,roads-area-text-name,roads-text-name,paths-text-name,railways-text-name,roads-text-ref-minor,text-poly-low-zoom,text-line,text-point,building-text,interpolation,addresses,water-lines-text,ferry-routes-text,admin-text,protected-areas-text,amenity-low-priority,text-low-priority
It seems to me that the SQL query in
amenity-line
:could be further simplified to:
?