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

Align line layers #85

Merged
merged 10 commits into from
Aug 3, 2024
Merged

Align line layers #85

merged 10 commits into from
Aug 3, 2024

Conversation

hiddewie
Copy link
Owner

@hiddewie hiddewie commented Aug 3, 2024

Part of #4

Align properties all railway line layers of zoom >= 8.

This makes data consumption easier, generates less tile data and reuses the same tile sources for all 5 layers.

Popups for railway lines will show the details of other visualizations as well (e.g. gauge is visible in the popup of the standard layer)

@hiddewie hiddewie temporarily deployed to proxy-production August 3, 2024 16:41 — with GitHub Actions Inactive
@hiddewie hiddewie temporarily deployed to proxy-production August 3, 2024 18:07 — with GitHub Actions Inactive
@hiddewie
Copy link
Owner Author

hiddewie commented Aug 3, 2024

Comparison of tile sizes (Europe)

Before:

image

After:

image

@hiddewie hiddewie marked this pull request as ready for review August 3, 2024 20:07
@hiddewie hiddewie merged commit c6617cf into master Aug 3, 2024
1 check passed
@hiddewie hiddewie deleted the align-line-layers branch August 3, 2024 20:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant