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

Aerialway pylons missing #822

Open
claysmalley opened this issue Mar 11, 2023 · 2 comments
Open

Aerialway pylons missing #822

claysmalley opened this issue Mar 11, 2023 · 2 comments
Labels
bug Something isn't working openmaptiles A change is needed in OpenMapTiles to support this

Comments

@claysmalley
Copy link
Member

claysmalley commented Mar 11, 2023

In #755, we added support for aerialway lines, but not for pylons, as they aren't present in OpenMapTiles:

Screenshot from 2023-03-11 17-07-36

Some map styles, like OSM-Carto, opt to style aerialway lines with an icon pattern that alludes to pylons, but the actual position of pylons is ignored:

Screenshot from 2023-03-11 17-08-12

Pylons are a useful visual reference when navigating urban areas on foot, and when represented on the map with a simple shape like a dot or square, they provide a pattern effect that complements aerialway lines. For reference, here's the same screenshot of OSM-Americana with crudely drawn dots representing aerialway=pylon and squares for aerialway=station:

Screenshot from 2023-03-11 17-07-36 (copy)

@claysmalley claysmalley added bug Something isn't working openmaptiles A change is needed in OpenMapTiles to support this labels Mar 11, 2023
@ZeLonewolf
Copy link
Member

That's amusing that osm-carto uses a pattern rather than the actual position. I agree it make sense to style the pylon nodes at the higher zooms where that level of detail makes sense.

@1ec5
Copy link
Member

1ec5 commented Mar 11, 2023

Do we expect to eventually symbolize aerialway stations? If so, we’d need to ensure that the pylon icons aren’t so prominent that the reader could mistake them for stations.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working openmaptiles A change is needed in OpenMapTiles to support this
Projects
None yet
Development

No branches or pull requests

3 participants