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 buildings with historic=castle as major #1176

Closed
matkoniecz opened this issue Dec 25, 2014 · 13 comments
Closed

render buildings with historic=castle as major #1176

matkoniecz opened this issue Dec 25, 2014 · 13 comments

Comments

@matkoniecz
Copy link
Contributor

Currently, after b4aff69 only buildings with amenity=place_of_worship are considered major. I propose to add also buildings with historic=castle as important tourism attractions and landmarks.

@Circeus
Copy link

Circeus commented Dec 27, 2014

I'm curious... What is the display difference between regular and major buildings?

@nebulon42
Copy link
Contributor

@Circeus
Copy link

Circeus commented Dec 28, 2014

So you mean that particular display style is linked to that aspect and not specifically to amenity=place_of_worship? I wouldn't have suspected (heck, I didn't know there was such a thing as "major buildings" in the style!).

@nebulon42
Copy link
Contributor

There wasn't until recently AFAIK (see #565 and #1153). The rendering is specific to amenity=place_of_worship but the question is apparently if this should be extended to other types of buildings as well. See also the discussion on the mailing list.

@matkoniecz matkoniecz added this to the Bugs and improvements milestone Dec 29, 2014
@matkoniecz
Copy link
Contributor Author

There is a problem - according to http://wiki.openstreetmap.org/wiki/Tag:historic%3Dcastle "draw a way around the outline of the construction.".

@pnorman
Is there a way to query whatever object is inside area tagged with historic=castle - without making it too inefficient?

@pnorman
Copy link
Collaborator

pnorman commented Jan 8, 2015

When I initially rewrote the buildings, I wanted to take out the special handling of buildings that had an amenity=place_of_worship tag because it didn't work well with churches that weren't P.O.W.s, churches as nodes inside buildings, and anything other than the simplest amenity=place_of_worship building=*.

We should probably decide on the above before looking at how to handle historic=castle, which is, if anything, even more complicated because the POI polygon contains the building polygon, not the other way.

@Wuzzy2
Copy link

Wuzzy2 commented Oct 3, 2016

Yes, please render castles, these are major landmarks.

@kocio-pl
Copy link
Collaborator

I think this is bad idea, so I will close it now. Icons should be much better and clear: #3099.

@dieterdreist
Copy link

dieterdreist commented Mar 26, 2018 via email

@kocio-pl
Copy link
Collaborator

I guess the icon and dark brown would be too much emphasis.

@dieterdreist
Copy link

dieterdreist commented Mar 26, 2018 via email

@kocio-pl
Copy link
Collaborator

I'm not sure if I understand you properly, but for city gates we have separate ticket (#152) and defensive towers are rendered now, so such popular objects would be visible anyway.

@dieterdreist
Copy link

dieterdreist commented Mar 26, 2018 via email

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

7 participants