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

Remove style for amenity=chalet #226

Closed

Conversation

CloCkWeRX
Copy link
Contributor

#204

I was less sure about the queries for this, as it wasn't immediately clear if they were tourism/leisure=chalet or amenity=chalet; so I did not modify the project.mml

@dieterdreist
Copy link

2013/10/14 CloCkWeRX [email protected]

I was less sure about the queries for this, as it wasn't immediately clear
if they were tourism/leisure=chalet or amenity=chalet; so I did not modify
the project.mml

+1 to remove amenity=chalet, there is not a single entity like this in the
db (according to taginfo). Docu says tourism=chalet.

@tyrasd
Copy link
Contributor

tyrasd commented Oct 17, 2013

Huh? This (4da4e3e) does only remove the chalet icon. Shouldn't we keep that and just fix this line to tourism=chalet?

@CloCkWeRX
Copy link
Contributor Author

Hrm may have missed a file when committing. ..
On 17/10/2013 7:54 PM, "Martin Raifer" [email protected] wrote:

Huh? This (4da4e3ehttps://github.com/gravitystorm/openstreetmap-carto/commit/4da4e3e1c7a57d7541ade0a38611ecf7f807d267)
does only remove the chalet icon. Shouldn't we keep that and just fix this
linehttps://github.com/gravitystorm/openstreetmap-carto/blob/master/amenity-points.mss#L78to
tourism=chalet?


Reply to this email directly or view it on GitHubhttps://github.com//pull/226#issuecomment-26490008
.

@CloCkWeRX
Copy link
Contributor Author

Closing for #238

@CloCkWeRX CloCkWeRX closed this Oct 20, 2013
@CloCkWeRX CloCkWeRX deleted the remove_amenity_chalet branch November 14, 2013 02:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new features Requests to render new features
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants