iD icon indicating copy to clipboard operation
iD copied to clipboard

Add support for roofs

Open egore opened this issue 12 years ago • 10 comments

It would be really helpful if iD would support setting the "roof:*" properties similar to how adresses can be edited on buildings.

From what I understood so far "address" is a special type of field. It would be a nice enhancement to allow these multi-value type of field definitions to be done in json as well.

If that would be in place it would be awesome to have: "roof:shape" as a "combo" "roof:orientation" as a checkbox for "along/across" "roof:height" as integer

egore avatar Sep 20 '13 13:09 egore

I would like to implement some improvements for maping buildings, including a way to map roof shapes more easily. I have something in mind that is similar to the graphical aproach of the app streetcomplete:

bildschirmfoto 2019-01-12 um 14 03 56

Where in the code is the edit menu for buildings defined? I would like to try a few things locally and then come back and make some suggestions.

stefangrotz avatar Jan 12 '19 13:01 stefangrotz

Where in the code is the edit menu for buildings defined? I would like to try a few things locally and then come back and make some suggestions.

Currently it's just defined as a field that fetches values from taginfo: https://github.com/openstreetmap/iD/blob/master/data/presets/fields/building.json

bhousel avatar Jan 12 '19 13:01 bhousel

Ah I see. Is it possible to put Icons as Labels in theese files or is this a text only field?

stefangrotz avatar Jan 12 '19 13:01 stefangrotz

Ah I see. Is it possible to put Icons as Labels in theese files or is this a text only field?

Currently text only, but adding support for icons would be useful. https://github.com/openstreetmap/iD/blob/master/modules/ui/combobox.js

bhousel avatar Jan 12 '19 13:01 bhousel

Does this preset files define wich fields are displayed in the menu? (/presets/presets!) https://github.com/openstreetmap/iD/blob/6ce2c5ea46b6bb89f8a3a3dd9d5794a920bbf610/data/presets/presets/building.json

stefangrotz avatar Jan 12 '19 13:01 stefangrotz

yes, check out the preset README here

bhousel avatar Jan 12 '19 13:01 bhousel

I added a Roof Color (roof:colour) field in the last release since it's a self-explanatory property that's readily visible in imagery. The other roof: tags would not make very user-friendly fields:

screen shot 2019-02-11 at 3 54 34 pm

quincylvania avatar Feb 11 '19 21:02 quincylvania

I was going to see about making a pull request to have the roof:shape common values as per the wiki in the iD editor, as it's annoying to have to use manual tagging so much when it has presets for some things (and sometimes then we get typos)

But the link above to the iD scheme json is broken - looks like iD may have been refactored since. Can someone point me/us/anyone to the current way to define a tag field in iD ?

trs998 avatar May 20 '25 09:05 trs998

@trs998 this is done in the id-tagging-schema Ideally this issue will be moved there.

tordans avatar May 20 '25 13:05 tordans

Started a pull request with what looks like the right tag values from the wiki - hopefully if not correct it'll at least get something moving. https://github.com/openstreetmap/id-tagging-schema/pull/1552

trs998 avatar May 20 '25 20:05 trs998