Skip to content

Update Removable Bollard string to make it clear the values includes bollards which are removable only with a key#1534

Open
andrewharvey wants to merge 4 commits intoopenstreetmap:mainfrom
andrewharvey:patch-4
Open

Update Removable Bollard string to make it clear the values includes bollards which are removable only with a key#1534
andrewharvey wants to merge 4 commits intoopenstreetmap:mainfrom
andrewharvey:patch-4

Conversation

@andrewharvey
Copy link
Copy Markdown
Collaborator

Without this it's unclear to mappers if this value is for removable by anyone who wants to go past or if it can also be for those removable bollards but which require a key or other token.

This is confirmed by the wiki description at https://wiki.openstreetmap.org/wiki/Key:bollard

…bollards which are removable only with a key

Without this it's unclear to mappers if this value is for removable by anyone who wants to go past or if it can also be for those removable bollards but which require a key or other token.

This is confirmed by the wiki description at https://wiki.openstreetmap.org/wiki/Key:bollard
@github-actions
Copy link
Copy Markdown

🍱 Your pull request preview is ready

Please use this preview to check your changes. Ideally use the test documentation template and document your test results by commenting on the PR. This will speed up the review process for everyone.

FYI, once this PR is merged, you can use the iD Editor Preview to test your changes in interaction with all other changes.

@tordans
Copy link
Copy Markdown
Collaborator

tordans commented May 23, 2025

This is what it looks like now

image

Info-i is not great.

image

I have added https://wiki.openstreetmap.org/wiki/Item:Q23021 which should be picked up.


It's not a huge change but I don't really like adding this kind of details to the string. This could start a lot of new changes that we don't want in this place. We have a better place for this which is the description of the field (learn more…). In openstreetmap/iD#10613 I am looking for ways to make the data added there more prominent.

My take is, to move this into the description and get an improved UI for those going …

@tordans tordans added field waitfor-consensus there seems to be no clear consensus on this in the osm communtiy; this has to wait labels May 23, 2025
@andrewharvey
Copy link
Copy Markdown
Collaborator Author

My take is, to move this into the description and get an improved UI for those going …

That's a good point. Let's go with that then, so we can close this PR.

https://wiki.openstreetmap.org/wiki/Item:Q23021

I'm not sure about this now.

"The bollard can be removed with our without tools."

With the right tools or enough force, anything can be removable. I think the intent is that it should be for bollards designed to be removed without force or destruction of the bollard, so that someone, either the public (no key needed) or the authorised person (those with a key) can easily remove it when needed then re-lock/place it.

Comment thread data/fields/bollard.json Outdated
@tordans
Copy link
Copy Markdown
Collaborator

tordans commented May 23, 2025

@andrewharvey I have no opinion on the wording. I just copied what I found on https://wiki.openstreetmap.org/wiki/Tag:barrier%3Dbollard#Types which I now see is very different to https://wiki.openstreetmap.org/wiki/Key:bollard

Feel free to change all that!

@tordans tordans removed the waitfor-consensus there seems to be no clear consensus on this in the osm communtiy; this has to wait label May 23, 2025
@matkoniecz
Copy link
Copy Markdown
Collaborator

Is this description supposed to appear somewhere? I cannot find it while testing.

BTW, probably "removable bollard" should be able to find at least basic bollard preset and it seems to fail now.

@matkoniecz
Copy link
Copy Markdown
Collaborator

matkoniecz commented Jan 2, 2026

How can I see changes caused by this PR in iD interface?

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants