Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

natural=shoal no longer rendered #3864

Open
Roburetto opened this issue Sep 3, 2019 · 6 comments
Open

natural=shoal no longer rendered #3864

Roburetto opened this issue Sep 3, 2019 · 6 comments

Comments

@Roburetto
Copy link

Roburetto commented Sep 3, 2019

Expected behavior (as it were until recently)

natural=shoal should depict areas as yellow
natural=shoal with surface=sand should depict areas as yellow with a sand pattern over

Actual behavior

natural=shoal shows nothing
natural=shoal with surface=sand shows a sand pattern over blue water

Links and screenshots illustrating the problem

natural=shoal
surface=sand
https://www.openstreetmap.org/way/699086566

natural=shoal
https://www.openstreetmap.org/way/699086565

Rendering suggestion

A possible rendering alternative could perhaps be to use transparency and place shoal areas under water as in the attached pic (where they are intuitively shown as transition areas in a beach - areas below coastline which became exposed at low tide).

shoal_sample

@imagico
Copy link
Collaborator

imagico commented Sep 3, 2019

That is intentional behavior - goes back to #3738.

There is discussion about some tuning of the appearance of shoal and beach outside the coastline - see #3840, but we definitely want to keep the land-water distinction as the primary visual distinction and not pretend that natural=shoal implies being above the high water line as it has unfortunately been the case previously.

The mapping recommendation - which is meant to be supported by the styling - is to accurately map the coastline at the top level of the regular tidal cycle.

@imagico
Copy link
Collaborator

imagico commented Sep 3, 2019

And we don't want to use transparency because of the confusing color mixing this creates in combination with other landcover colors. See #3854.

@jeisenbe
Copy link
Collaborator

jeisenbe commented Sep 4, 2019

If the technical changes in #3854 are implemented, then it would be possible to add a subtle rendering for shoals which lack a surface= tag, perhaps a slightly darker and less saturated blue fill, similar to that achieved for tidalflat over water.

Another option would be to use a generic-looking pattern for shoals that lack a surface tag, perhaps reef.svg since there isn't a very clear distinction between when a feature should be mapped as a reef or a shoal - in most shoals are sand and gravel features, and most reefs are rock or coral features, but the wiki definitions do not require this and seem to overlap in some cases.

I would recommend that mappers consider adding the surface= tag when possible, since a shoal can be sand, gravel or solid rock, and these could be rendered with different patterns. But if the material of the shoal isn't known, then it's fine to map it without the surface tag, and we should try to accommodate this tagging with the rendering.

@georgek
Copy link

georgek commented Sep 18, 2019

Since it is due to the same issue I should add that natural=bare_rock isn't rendering any more either.

@jeisenbe
Copy link
Collaborator

For natural=bare_rock see #3851

@mfrasca
Copy link

mfrasca commented Apr 11, 2021

in the Caribbean, wherever there's coral reefs, we have shoal working as shallow natural wavebreakers, definitely not man_made. it's also not a tidalflat because there's hardly any tides here, less than the 50cm we have in the Mediterranean. we can include it in the mapping of reef, but it's so shallow you can't even swim there. and it's areas, not nodes, so shoal with surface=* sounds more fitting than marking a row of dangerous rocks with nodes. one would expect it to be prominently rendered, and it's not practical that it would not show at all.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants