-
Notifications
You must be signed in to change notification settings - Fork 822
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
dog_park base and pattern colors clash with general color system in this style #2250
Comments
2016-07-27 10:39 GMT+02:00 Christoph Hormann notifications@github.com:
dog parks are (at least in this area) normal parks, usually fenced areas of |
Note the examples i linked to are specifically cases where these are not parts of a regular park. Looking over occurrences it seems separate dog areas and parts of regular parks are similarly common. |
Wow, that's strong. |
It's nice that dog parks get rendered - that's something that can show off the data of OSM. But I agree with all points above, the current rendering is not working.
|
I've been playing around and came up with this: The new playground color is used (on the grounds that dog parks are playgrounds for dogs, and its a good enough color anyway). The shop_pet icon is reused in the pattern. I don't think the new paw icon is significantly different from the shop_pet one we already have (shop_pet on the left, current paw pattern on the right). It also seems like dog parks mapped as nodes should be in amenity-brown for the same reason playgrounds are. @daganzdaanda, I'm afraid having a green "site" border will make it look like a nature reserve. |
@meased Could you create a PR for testing? |
See also discussion in #2216.
The base color
The symbol color
Examples:
http://www.openstreetmap.org/#map=15/48.1231/7.8258
http://www.openstreetmap.org/#map=16/48.7070/9.1655
http://www.openstreetmap.org/#map=16/47.4295/8.5128
The text was updated successfully, but these errors were encountered: