Skip to content
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

transparent symbol for private recycling and playgrounds #1191

Merged
merged 1 commit into from
Jan 13, 2015
Merged

transparent symbol for private recycling and playgrounds #1191

merged 1 commit into from
Jan 13, 2015

Conversation

matkoniecz
Copy link
Contributor

implements part of #1012

Thanks to @nebulon42 for SVG icons.

The same method as for private parking, without considering access=public as synonymous to access=yes

@nebulon42
Copy link
Contributor

Private parking also changes text colour, maybe this should be done for playground too?

@matkoniecz matkoniecz self-assigned this Jan 4, 2015
@matkoniecz
Copy link
Contributor Author

Good idea, I will do this.

@matthijsmelissen
Copy link
Collaborator

I'm not sure if we want to render an icon for private playgrounds and recycling centre at all? Also I don't know if people will understand the meaning of this use of transparency at all?

@polarbearing
Copy link
Contributor

Why not, I think it is good to see if e.g. a school or a residential structure has a playground or not.
And yes, they will understand as long as there is consistency (private parking, playing, museum, etc)

@matkoniecz matkoniecz removed their assignment Jan 11, 2015
@matthijsmelissen
Copy link
Collaborator

I don't think the new text colour looks nice. Perhaps there is a better colour? I also wouldn't mind keeping the text the same.

@matkoniecz
Copy link
Contributor Author

access private leisure playground name eeeeee eeeeee eeeeee ref 1 ele 8000 closed_way master - private_transparent

What about this one?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants