gravitystorm / openstreetmap-carto

A general-purpose OpenStreetMap mapnik style, in CartoCSS
Other
1.52k stars 815 forks source link

Please render natural=shingle in river #3894

Open ghost opened 4 years ago

ghost commented 4 years ago

Currently, areas of natural=sand in a river are rendered, but areas of natural=shingle aren't. That's not consistent. Could you please render natural=shingle in a river, too?

Untitled

Screenshot is from here.

jeisenbe commented 4 years ago

Thank you for mentioning this. We have recently been discussing how to render areas of bare_rock in the intertidal zone and in rivers, and natural=shingle is also another feature that could be shown in the same way.

Note that natural=beach with surface= currently renders the pattern above water areas, but I agree that not all areas of shingle along a river can be tagged as natural=beach

(Also natural=sand should not render above water. Areas of sand formed by moving water along rivers, lakes and seas should be tagged natural=beach)

Related issues: #3707, #3840, #3864, #3877 and especially #3854 Related PRs: #3855 and #3851

ghost commented 4 years ago

I'm unsure whether areas of shingle or sand at a river borders can be considered beaches because they often have a very different shape than areas of shingle or sand at lakes or seas due to the lateral instead of a vertical movement of the water. Example (location):

Untitled