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

Remove area type for traffic_calming=island #1076

Merged
merged 2 commits into from
Feb 28, 2024
Merged

Conversation

ireun
Copy link
Contributor

@ireun ireun commented Dec 21, 2023

Hello!

This is yet another request to make Traffic Calming Islands great again! 😆

Thing is that traffic_calming=* is not meant to be used as areas, and so is not traffic_calming=island.

At least since Nov 2020 iD has been suggesting traffic_calming=island for areas under a misleading name 'Traffic island' (this has been fixed in #1074). Probably some of the mistakenly tagged islands were created because of that iD preset. For areas area:highway=traffic_island should be used instead.

There are currently 40k usages of that tag in combination with area=yes: https://taginfo.openstreetmap.org/tags/traffic_calming=island?filter=ways#combinations

I would also argue to restrict area usage for all traffic_calming=* - Though I'd like to think about that for a bit. :)

Wiki references:

Copy link

🍱 You can preview the tagging presets of this pull request here.

@ireun
Copy link
Contributor Author

ireun commented Dec 21, 2023

Also I see that iD warns about usage of traffic_calming=island on ways. Not sure why though nor how to fix that.
image

@tyrasd tyrasd added this to the v6.6 milestone Jan 17, 2024
@tyrasd tyrasd added the bug Something isn't working label Jan 17, 2024
@tyrasd tyrasd modified the milestones: v6.6, v6.7 Jan 17, 2024
@tyrasd
Copy link
Member

tyrasd commented Jan 17, 2024

Also I see that iD warns about usage of traffic_calming=island on ways. Not sure why though nor how to fix that.

This is caused by openstreetmap/iD@dc9b6a3 🙈 unfortunately

In order to avoid these false warnings, I'll have to fix that on iD side first, before we can merge this PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants