-
Notifications
You must be signed in to change notification settings - Fork 823
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
Add special rendering for shop=jewelry #1415
Comments
this should be shop=jewellery because all tags are in British English |
It's still stuck at the proposition level: http://wiki.openstreetmap.org/wiki/Proposed_features/BE-Spelling_shop%3Djewelry The usage is like 15996 vs 170, so:
I would pick 1 for now and maybe 2 when proposal is accepted. |
Or we can wait with prominent rendering until situation is fixed - promoting better tagging scheme is OK but not in situation when it is losing 1 to 100. |
Sure. I said what I like the most, but for me any option is acceptable, since all have their strengths and weaknesses. |
Yes - but what it should be is a tagging issue, while what it is is fairly clear. |
2015-03-20 0:20 GMT+01:00 Paul Norman [email protected]:
yes, but promoting an AE spelling variant by bringing it on the map will |
We already do render railway=subway rather than UK English railway=metro or railway=underground. In UK English, a subway is a pedestrian underpass AFAIK. Other common US English tags are highway=elevator (should be highway=lift), and sport=soccer (should be sport=football). I'm sure there will be more. |
What's missing for this to be done? A style patch only? |
On my list. But as I wrote in #1402 (comment) I'm waiting for maintainers approvement as I don't want to end up with a lot of PRs that have no chance of getting merged. |
…t, jewelry, mobile_phone and optician (resolves gravitystorm#1415 gravitystorm#1412 gravitystorm#1408)
…t, jewelry/jewellery, mobile_phone and optician (resolves gravitystorm#1415 gravitystorm#1412 gravitystorm#1408)
…t, jewelry/jewellery, mobile_phone and optician (resolves gravitystorm#1415 gravitystorm#1412 gravitystorm#1408)
We should start rendering shop=jewelry, probably with this icon from osmfr-cartocss fork:
https://github.com/cquest/osmfr-cartocss/blob/master/symbols/jewelry.svg
[It was proposed in meta-issue #1402]
The text was updated successfully, but these errors were encountered: