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

Render name=* on landuse=religious #4999

Closed
BorutAtGit opened this issue Aug 3, 2024 · 3 comments
Closed

Render name=* on landuse=religious #4999

BorutAtGit opened this issue Aug 3, 2024 · 3 comments

Comments

@BorutAtGit
Copy link

Expected behavior

name=* for landuse=religious shown (parhaps at , or above zoom levels 15-16)

Actual behavior

name=* for landuse=religious not shown at any zoom level

Screenshots with links illustrating the problem

https://www.openstreetmap.org/way/802140932#map=18/41.24871/21.60784

###Background
If I am not wrong, Carto is currently not showing name=* for landuse=religious at any zoom level. I am currently traveling through N.Macedonia. There are quite a lot of monasteries tagged with that combination arround, but apparently their names are not shown. Thanks!

@imagico
Copy link
Collaborator

imagico commented Aug 3, 2024

Thanks for the report.

landuse=religious is indeed not rendered with a name label. There is dead code in amenity-points.mss for this - but the corresponding SQL query does not include landuse=religious features.

I have doubts that adding name label rendering for landuse=religious is a good idea for us. In most cases where a name is tagged (which is only around 21 percent) - including this one - the name refers to a more specific feature. In this case that would be the monastery (amenity=monastery) - for which we have an open issue #3036. I would prefer us rendering specific religious features like that with a name label rather than generically landuse=religious.

@imagico imagico added the text label Aug 3, 2024
@HolgerJeromin
Copy link
Contributor

This was reported in #4214

@imagico
Copy link
Collaborator

imagico commented Aug 3, 2024

Indeed, thanks @HolgerJeromin - closing as duplicate.

@imagico imagico closed this as not planned Won't fix, can't repro, duplicate, stale Aug 3, 2024
@imagico imagico added duplicate and removed text labels Aug 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants