Skip to content

Add rendering of aerialway=zip_line #3156

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

Closed
jragusa opened this issue Mar 30, 2018 · 7 comments
Closed

Add rendering of aerialway=zip_line #3156

jragusa opened this issue Mar 30, 2018 · 7 comments

Comments

@jragusa
Copy link
Contributor

jragusa commented Mar 30, 2018

Expected behavior

Basically, aerialway=zip_line should be rendered like most of aerialways (e.g. aerialway=chair_lift) Using a distinct pattern would be excluded due to the low usage (only 426 on Taginfo with regard to the 8053 counts for aerialway=chair_lift).

Actual behavior

aerialway=zip_line is not rendered.

Links and screenshots illustrating the problem

z=16 (https://www.openstreetmap.org/way/132119392)
aerialway_zip_line_z16
Please note the two stations "Fantasticable" to the north that each correspond to a aerialway=zip_line

@jragusa
Copy link
Contributor Author

jragusa commented Mar 30, 2018

Two additional remarks: aerialway=magic_carpet is also not displayed (1738 usages) and the rendering of aerialway=mixed_lift is missing on the wiki page.

@HolgerJeromin
Copy link
Contributor

Magic carpets are mobile things (removed after snow) for winter sport only. I would not recommend rendering in this map style.

@kocio-pl kocio-pl added this to the Bugs and improvements milestone Mar 31, 2018
@kocio-pl
Copy link
Collaborator

If they are seasonal, maybe they should be rendered wit a dashed line, just like intermittent waterways?

@HolgerJeromin
Copy link
Contributor

That is the problem when mixing issues.

I expect ziplines as non seasonal.

@jragusa
Copy link
Contributor Author

jragusa commented Mar 31, 2018

Some zip_line are seasonal but most of them are available during both winter and summer.

@jragusa
Copy link
Contributor Author

jragusa commented Apr 5, 2018

If it's ok for everyone I can make a PR for this issue. It's just 1 line to add in aerialways.mss

@kocio-pl
Copy link
Collaborator

kocio-pl commented Apr 5, 2018

OK, do it please.

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