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

add rendering for amenity=doctors #1201

Merged
merged 1 commit into from
Jan 13, 2015

Conversation

nebulon42
Copy link
Contributor

Implements #89 and part of #1165. Supercedes #1041.
SVG Icon: nebulon42/osmic/health/doctor-16.svg

http://www.openstreetmap.org/#map=19/48.21663/16.34037
before
not rendered

after
doctors_after

@nebulon42 nebulon42 mentioned this pull request Jan 6, 2015
66 tasks
@matthijsmelissen
Copy link
Collaborator

This icon is interpreted as a pharmacy (n=1).

@daganzdaanda
Copy link

I see the potential to read it as "pharmacy", but it can quickly be learned, and makes a good icon.
The other option at #1041 was a snake of Aesculap, which is well known, but horrible at those small sizes.

@nebulon42
Copy link
Contributor Author

I this might be possible, yes, but I also think that in combination with the name and the very expressive pharmacy icon this is easily spotted as doctors.

matthijsmelissen added a commit that referenced this pull request Jan 13, 2015
add rendering for amenity=doctors
@matthijsmelissen matthijsmelissen merged commit 3aafa78 into gravitystorm:master Jan 13, 2015
@nebulon42 nebulon42 deleted the svg-doctors branch January 13, 2015 22:07
This was referenced Jan 13, 2015
@matthijsmelissen
Copy link
Collaborator

@nebulon42 Can you include the phrase 'Closes #XXX' or 'Resolves #XXX' in your PR description? That way the issue automatically gets resolved when the PR is accepted.

@nebulon42
Copy link
Contributor Author

Yes, sure.

@daganzdaanda
Copy link

Thank you all, finally we see doctors on the map! And it looks good!

I've found an extreme place: https://www.openstreetmap.org/relation/2128448
The doctors should probably be moved inside, but then it would be OK for me.

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

Successfully merging this pull request may close these issues.

3 participants