Opened 9 years ago
Closed 9 years ago
#12337 closed enhancement (fixed)
add support for shop=tobacco
Reported by: | dieterdreist | Owned by: | team |
---|---|---|---|
Priority: | normal | Milestone: | 16.02 |
Component: | Internal preset | Version: | |
Keywords: | tobacco, shop | Cc: | zermes |
Description
the tag is currently used 4444 times and it is documented in the wiki.
Attachments (1)
Change History (11)
follow-up: 2 comment:1 by , 9 years ago
comment:2 by , 9 years ago
Component: | Core → Internal preset |
---|---|
Milestone: | → 16.02 |
comment:3 by , 9 years ago
Cc: | added |
---|
black & white icons are not great for JOSM because of our black background. zermes, any idea? :)
comment:4 by , 9 years ago
yes, I didn't mean this literally, I drew the shape template so you can change the colour to any colour that integrates well with the other shop icons. If you tell me the colour code I can do it if you prefer this. There is no white in the icon, just one colour. Or if a border is desired. You can also point me to a link to another svg shop icon so I can copy these things from there.
comment:5 by , 9 years ago
There is no color code for shop icons. Just make sure that the icon is visible atleast on black, white and grey background.
We cannot use cc-by icons, because cc-by is not compatible with GPL.
more icon guidlines: https://josm.openstreetmap.de/wiki/DevelopersGuide/DefaultPresets#NewIcons
by , 9 years ago
Attachment: | tobacco-pipe.svg added |
---|
tobacco pipe to be used for shop=tobacco. License cc0, created by Martin Koppenhoefer
comment:6 by , 9 years ago
thank you for the hints, I have now replaced the icon with a (hopefully correctly formated) updated version and changed the license to cc0.
comment:7 by , 9 years ago
Thanks. I'm unsure in which group this belongs: "For the body" or "Other"?
...or "Food" ;)
comment:8 by , 9 years ago
yes, "For the body" seems a good fit, maybe you should also move the funeral directors from "other" to this category ;-)
comment:9 by , 9 years ago
btw.: I'm unsure if I have used the correct width and height, I made this now 100% and the viewBox as suggested 0 0 16 16, if it doesn't work like this it must be 16 width and height.
if needed I can provide an icon.