-
Notifications
You must be signed in to change notification settings - Fork 821
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
aerialway=magic_carpet #4650
base: master
Are you sure you want to change the base?
aerialway=magic_carpet #4650
Conversation
Ref #4649 (duplicate of #3169 (comment)). |
Thanks for the pull request. #3169 has been closed with the decision not to render this - this would need to be revised to accept this change. Apart from that it seems most of these features are small, the vast majority less than a hundred meters in length. That would mean at the starting zoom level (z12) a length of just a few pixels in most cases. |
I suggest to have discussion if or if not to render this on #3169. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In my eyes this will require more work than just adding this as yet another aerialway type - mostly because aerialway=margic_carpet
is different in meaning to the map user, both those who might want to use it in its function and those who see this merely as an artefact/obstacle.
As written in #3169 (comment) - if you want to work on that it would be important to consider the following:
- consider the typical size of these in rendering.
- consider that these are very different in meaning both to the user (in winter) and when it is not operational (as an obstacle) compared to actual aerialways in the true sense of the term.
- consider that rendering these but not rendering
highway=footway
+conveying=yes
in a distinct fashion would probably lead to a problematic mapping incentive.
I'd encourage you to work on this but be aware that this will require both more coding and more actual design work. Especially finding a good solution to display highway=footway
+ conveying=yes
could be quite challenging.
Fixes # 4649
Changes proposed in this pull request:
Test rendering with links to the example places:
https://www.openstreetmap.org/#map=18/40.32955/-7.61252
In this place there is a magic carpet
Before
After