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.

Sign up for GitHub

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

Jump to bottom

Vending machines rendering at all zoom levels #3623

Closed
meased opened this issue Jan 6, 2019 · 3 comments · Fixed by #3624
Closed

Vending machines rendering at all zoom levels #3623

meased opened this issue Jan 6, 2019 · 3 comments · Fixed by #3624
Labels
amenity-points bug
Milestone
Bugs and improvem...

Comments

@meased
Copy link
Contributor

meased commented Jan 6, 2019

The recently committed #3601 is rendering vending machine icons at all zoom levels.

vending

Apparently there is a missing zoom filter in amenity-points.mss line 190.
@matthijsmelissen
Copy link
Collaborator

@jragusa Would you be able to look into that?

@kocio-pl kocio-pl added this to the Bugs and improvements milestone Jan 6, 2019
@kocio-pl
Copy link
Collaborator

kocio-pl commented Jan 7, 2019

Until this bug is fixed, new release (#3604) is going to be blocked.

@jragusa
Copy link
Contributor

jragusa commented Jan 7, 2019 •
edited
Loading

oh god, I completely forgot to add zoom level in amenity-points.mss

I reorganised lines about icons but I forgot to add zoom level

My apologises for the disturbance. I'm pushing the bugfix.

@jragusa jragusa mentioned this issue Jan 7, 2019
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
amenity-points bug
Projects
None yet
Milestone
Bugs and improvements
Development

Successfully merging a pull request may close this issue.

[Bugfix] Add zoom level to vending_machine
4 participants
@matthijsmelissen @kocio-pl @jragusa @meased