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:
Move
leisure=track
andattraction=water_slide
fromamenity-line
tolandcover-line
layer[Also tweak to
highway=ford
SQL query to simplify hstore query]Test rendering with links to the example places:
Before
Power line vs. track ordering
After
Unchanged renders for:
Ford as linear way
attraction=water_slide
Comments
As a ground-level feature,
leisure=track
is a very natural fit for early rendering inlandcover-line
. It's less obvious that water slides should be rendered e.g. before roller-coasters. Perhaps a future optimisation would be to merge many of these layers, so that thelayer
tag can be properly exploited?[Note that #4104 proposes moving
leisure=track
to the roads layer (tricky) oraerialways
andattraction=water_slide
towaterway-bridges
. It would make sense to consider these alternatives.]