-
Notifications
You must be signed in to change notification settings - Fork 63
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 places of worship #910
Conversation
PR Preview: Sprite Sheets: |
Thanks for taking this on! There was a Slack discussion that discussed PoW icons and in summary I think we need to:
There was also some discussion about modifying the base rectangle size if needed for effective size contrast between the symbol and the base. |
Updated the PR to color the symbols blue and add thicker lines for the Star of David. I'll continue iterating the symbol relative sizes via slack, though I can also post the test screenshots here if anyone desires |
}, | ||
sprite: "poi_pow_christian", | ||
color: Color.poi.infrastructure, | ||
description: "Christian place of worship", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
id-tagging-schema calls this tagging combination simply “Christian church”, and there are other religion-specific descriptions for other religion
values.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm open to this, and considered it for the initial PR, though I worried it gets into complicated terminology issues quickly. For instance, not all amenity=place_of_worship
+ religion=christian
are churches (like LDS Temples or even holy wells), some sects of Judaism call their houses of worship 'temples' or 'shuls' instead of synagogues, what iD labels as "Sikh temples" are actually called gurdwaras, etc. But maybe these are small potatoes compared to the improvement in how understandable the legend would be.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That’s true, your point probably matters more in the context of a public-facing map like Americana than something internal to OSM like iD. The diplomatic “place of worship” terminology isn’t inaccurate, just wordy, but we have enough room for it.
Ah thanks for going ahead and merging, I've been busy with work lately and haven't had much time to work on this. I think as some of the comments here and on Slack have noted, if it turns out "in the wild" that the religious symbols are too small, it could potentially be useful to make the squares slightly smaller and the icons slightly bigger. I'll also open separate issues to track other potential religions. |
Fixes #861.
This PR adds places of worship. The icon is based on the classic US atlas style of a square topped with a religious icon. The initial PR contains the places of worship of the 8 religions given an icon in Carto: Buddhist, Christian, Hindu, Jewish, Muslim, Sikh, Shinto, and Taoist. The symbols I chose are:
Places of worship do not necessarily fall neatly into any of the defined POI color categories. The initial PR uses attraction-brown, but there were suggestions in slack discussions for infrastructure-blue or perhaps its own color (maybe gray?). I'm open to suggestion on colors.
Other religions could be considered. Some other common
religion
values in the United States includeunitarian_universalist
(518place_of_worship
) andbahai
(79). This PR also lacks distinct icons for LDS and Jehovah's Witness places of worship, which are rendered with crosses. Distinct renderings are blocked by OMT, which does not contain anydenomination
information that could distinguish these from other Christian POWs.Samples:
Christian and Hindu (localhost link):
Buddhist, Christian, Jewish, and Shinto (localhost link):
Christian and Muslim (localhost link):
Buddhist, Christian, and Taoist (localhost link):
Christian and Sikh (localhost link):
Legend sample:
I opted for the uniform "place of worship" rather than attempting to name each religion's.
Jerusalem Temple Mount (localhost link):