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

Buildings overlay - support editing of buildings with building:use tag #5506

Open
davidpnewton opened this issue Feb 26, 2024 · 6 comments
Open
Labels
enhancement help wanted help by contributors is appreciated; might be a good first contribution for first-timers

Comments

@davidpnewton
Copy link

davidpnewton commented Feb 26, 2024

There are considerable numbers of buildings that the new Buildings overlay is not picking up as such and thus not allowing the editing of.

Look at the buildings in the French commune of St Etienne au Mont just to the south east of Boulogne-sur-Mer to see examples of this.

How to Reproduce

Compare way 132262690 with way 132264085 for an example of two buildings directly next to each other. Difference appears due to something with building levels or building roof type tagging.

Expected Behavior
Both way 132262690 and way 132264085 picked up as buildings by the overlay and thus editable.

Versions affected
Android 14
Streetcomplete 57.0 beta-2

@matkoniecz
Copy link
Member

https://www.openstreetmap.org/way/132262690

https://www.openstreetmap.org/way/132264085

buildings with building:use keys currently are skipped:

@matkoniecz matkoniecz changed the title Buildings overlay failing to make many buildings editable Buildings overlay is not enabling editing of buildings with building:use tag Feb 26, 2024
@matkoniecz matkoniecz changed the title Buildings overlay is not enabling editing of buildings with building:use tag Buildings overlay - support editing of buildings with building:use tag Feb 26, 2024
@mnalis
Copy link
Member

mnalis commented Feb 26, 2024

Makes sense to me. Unless quest is going to be extended to handle such cases (with special subquestions, confirmations, or whatever), this should probably be added to SC FAQ as it sounds confusing.

@westnordost
Copy link
Member

westnordost commented Feb 27, 2024

It is pretty much a missing feature to reduce implementation complexity of that overlay.

If this is implemented (by someone), "historic", "ruins", "disused" (etc?) should be handled accordingly. E.g.

building=industrial + building:use=apartments -> "Built as industrial building, now it is apartment building"
building=industrial + ruins=yes -> "Built as industrial building, now it is ruins"

I do not intend to work on it in the matter (I put !building:use in the filter deliberately after all) in the foreseeable time because I want to defer work that concerns the UI to after it has been migrated to Compose

@matkoniecz matkoniecz added the help wanted help by contributors is appreciated; might be a good first contribution for first-timers label Mar 1, 2024
@call-me-matt
Copy link

call-me-matt commented Mar 26, 2024

I understand and agree to ignore complex buildings. But for a better user experience, I suggest to give it a (greyed out) color code. As it is now, the user thinks the building tag is not correct or the map data not up to date. This triggers experimenting with the data which is contra-productive.

as-is:to-be:

@westnordost
Copy link
Member

Experimenting? You cannot, as you cannot click on it, or can you?

@call-me-matt
Copy link

call-me-matt commented Mar 26, 2024

Well, you are right. But when buildings do not show up in StreetComplete, people use other editors to "fix" the building that seems to be not correctly mapped. I have seen it at different locations. I was also confused and searched here before understanding the logic behind. In my point of view a color code can improve the user experience here.

Edit:
I just realized that you have already an overlay for "other" buildings. Why not use that one in the meantime for building:use tags until this issue is closed?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement help wanted help by contributors is appreciated; might be a good first contribution for first-timers
Projects
None yet
Development

No branches or pull requests

5 participants