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

Road shield blocks highway=mini_roundabout #4167

Closed
Prince-Kassad opened this issue Jun 21, 2020 · 2 comments · Fixed by #4218
Closed

Road shield blocks highway=mini_roundabout #4167

Prince-Kassad opened this issue Jun 21, 2020 · 2 comments · Fixed by #4218

Comments

@Prince-Kassad
Copy link

Example illustrating the problem:

https://www.openstreetmap.org/#map=19/49.15015/7.77923&layers=N

highway=mini_roundabout should have priority over everything else since it's a road feature, yet the road shield can block it.

mini_roundabout

@imagico
Copy link
Collaborator

imagico commented Jun 22, 2020

Yes, mini roundabouts are rendered as low priority blocking POIs.

A possible solution would be to rendering this in a non-blocking fashion immediately after the roads layers. Would need to be tested for feasibility though.

Another option - which i think i'd kind of favor - would be to render them like highway=turning_circle, maybe with a minor design variation to indicate the difference in tagging - see

https://wiki.openstreetmap.org/wiki/Tag:highway%3Dturning_circle
https://wiki.openstreetmap.org/wiki/Tag:highway%3Dmini_roundabout

Physically these are very similar and the difference largely comes from context - which is visible in the map.

@jeisenbe
Copy link
Collaborator

jeisenbe commented Sep 1, 2020

Re: "render them like highway=turning_circle, maybe with a minor design variation to indicate the difference in tagging"

I agree with this idea. Physically they are nearly identical to turning circles, so including a minor design variation seems not strictly necessary, though if it's possible to render a small circular gray dot in the center that could look nice.

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

Successfully merging a pull request may close this issue.

3 participants