Skip to content

Add Barrack Building as a value for building type dropdown#2171

Open
ak8abhinay wants to merge 3 commits intoopenstreetmap:mainfrom
ak8abhinay:add-building-barracks
Open

Add Barrack Building as a value for building type dropdown#2171
ak8abhinay wants to merge 3 commits intoopenstreetmap:mainfrom
ak8abhinay:add-building-barracks

Conversation

@ak8abhinay
Copy link
Copy Markdown
Contributor

Adds new preset for building=barracks

wiki : https://wiki.openstreetmap.org/wiki/Tag:building%3Dbarracks

Fixes #2156

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 9, 2026

🍱 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.

@ak8abhinay
Copy link
Copy Markdown
Contributor Author

Here is a screenshot of the preset working in the iD editor.

image

Building type dropdown :

image

@matkoniecz matkoniecz added new-label changes the name, aliases or terms of a preset new-preset labels Apr 9, 2026
@matkoniecz
Copy link
Copy Markdown
Collaborator

I am not sure yet should it get own building preset or is dropdown in building type enough.

But tent icon is definitely bad.

@ak8abhinay
Copy link
Copy Markdown
Contributor Author

I am not sure yet should it get own building preset or is dropdown in building type enough

Even I wasn't sure initially. But after looking at the building types like Yurt which has its own preset as well as dropdown and even statistics wise where only 944 is the tag usage or Hanger Building , pigsty which have both its own preset and dropdown as well.

But tent icon is definitely bad

Agree.
Another option we have is :

image

@matkoniecz
Copy link
Copy Markdown
Collaborator

Problem with barracks is that distinct building type is quite dubious and there is much greater risk of people misusing it for the current use. In fact the second icon (with flag) would make it even worse: if soldiers leave and flag is removed then it should be still tagged building=barracks. Or it should not be tagged as building=barracks in the first place.

Having it as dropdown of building type makes it more clear that it is about building type, not about building use.

In fact, do we have preset for things currently used as barracks?

@matkoniecz matkoniecz added the waitfor-requested-changes This PR is waiting for an update to incorporate requested changes from a PR review. label Apr 11, 2026
@ak8abhinay
Copy link
Copy Markdown
Contributor Author

do we have preset for things currently used as barracks?

military=barracks already exists as a preset in iD.

I totally agree the existing preset covers the situation where one can map an active military barracks and if wants to tags the building on top of that can be best served by the dropdown.

@ak8abhinay ak8abhinay changed the title Add building=barracks preset and as a value for building type dropdown Add Barrack Building as a value for building type dropdown Apr 11, 2026
Comment thread data/fields/building.json Outdated
Co-authored-by: Mateusz Konieczny <matkoniecz@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

new-label changes the name, aliases or terms of a preset new-preset waitfor-requested-changes This PR is waiting for an update to incorporate requested changes from a PR review.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

add support for building=barracks

2 participants