openstreetmap / id-tagging-schema

🆔🏷 The presets and other tagging data used by the iD editor
ISC License
137 stars 148 forks source link

Create preset for footwear_decontamination #1235

Open k-yle opened 1 month ago

k-yle commented 1 month ago

A preset was recently added for amenity=bicycle_wash - see #1032.

This PR creates a preset for the similar approved tag man_made=footwear_decontamination.

It is currently limited to Australia and New Zealand, since I'm not aware of anywhere else in the world where these features exist. However, I see that man_made=carpet_hanger (#452) is a global preset, despite only existing in a very small number of countries. So maybe it's okay to make this a global preset?

github-actions[bot] commented 1 month ago

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

tordans commented 1 month ago

I think starting with presets locally is better. It makes it easier to have a good UX when less relevant options are not present for other countries (and easy to fix later). I would rather think we should limit the other preset to countries that are relevant.

tordans commented 1 month ago

@k-yle looking at our new draft docs, I wonder if this is merge ready or fits our criteria already. A few thoughts…

matkoniecz commented 1 month ago

However, I see that man_made=carpet_hanger (https://github.com/openstreetmap/id-tagging-schema/pull/452) is a global preset, despite only existing in a very small number of countries.

Maybe man_made=carpet_hanger should become limited?

screen12 screen13

k-yle commented 1 month ago

Icon: [...]

I've opened https://github.com/rapideditor/temaki/pull/93 to create a dedicated icon, so I suppose this PR needs https://github.com/openstreetmap/id-tagging-schema/labels/waitfor-icon

Usage: It's only 160 elements ATM which is really low given #1229 (files). However, its also a regional preset, so that does not have to be a blocker. [...]

It is low, but it's higher than amenity=bicycle_wash when it was added, and that one is a global preset. That's the main reason why I thought this would be acceptable.

Currently, the fallback preset is Man-Made Feature which doesn't really help beginners to find the tag, even I don't map them sometimes, because it's so hard to search the wiki for a specific tag. So a preset would help everyone who maps trails/forests/bush areas

tordans commented 3 weeks ago

Is this ready to merge from a technical point of view? @k-yle @tyrasd ?

I resolved conflicts that happened after merging #1237 and updated the icons.json that was generated during one of the npm run commands. The is released on https://rapideditor.github.io/temaki/docs/ so that should work as well.

I am wondering where the updated icon will be fetched. Temaki is not in the package.json of this repo and not in the schema builder repo. So it looks like the consumers of this repo need to handle adding the icon libraries themselves like in id.

So from my point of view this can be merged. Any objections?

k-yle commented 3 weeks ago

Is this ready to merge from a technical point of view?

I believe so, but iD will need to update temaki before the next release. I was hoping @dependabot would do it, but the bot seems to limit itself to 5 open PRs. So I've created https://github.com/openstreetmap/iD/pull/10280

I am wondering where the updated icon will be fetched. [...]

Yes, in iD's case, running npm run dist:svg:temaki will bundle every icon from temaki into a single file:
dist/img/temaki-sprite.svg, which is loaded when you open iD