Skip to content

style: show oneway road arrows [#499] #501

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

Merged
merged 3 commits into from
Jul 4, 2025
Merged

style: show oneway road arrows [#499] #501

merged 3 commits into from
Jul 4, 2025

Conversation

bdon
Copy link
Member

@bdon bdon commented Jun 5, 2025

No description provided.

@wipfli
Copy link
Collaborator

wipfli commented Jun 6, 2025

The arrows should have a tail - something like this:

image

On my not-so-great laptop display it is otherwise too hard to see the direction with the current sprite

image

@bdon
Copy link
Member Author

bdon commented Jun 6, 2025

we should make the arrow bigger at all zooms, but that would also require changing the road widths to be sufficiently wide

see http://tangrams.github.io/bubble-wrap/#18.20833333333332/37.79213/-122.40090 for comparison

If the arrows are that size do you think we still need the tail?

@wipfli
Copy link
Collaborator

wipfli commented Jun 6, 2025

I think scaling normal raster png sprites is bad. I made the size of the original arrow now 0.5 times smaller and for me it looks OK. Let me know what you think.

Lake area in Zürich

image

@bdon
Copy link
Member Author

bdon commented Jun 7, 2025

Looks good!

Copy link

@wipfli
Copy link
Collaborator

wipfli commented Jun 26, 2025

CI is green now @bdon

@bdon bdon merged commit 4b48430 into main Jul 4, 2025
6 checks passed
@bdon bdon deleted the style-oneway-arrows branch July 4, 2025 03:54
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.

2 participants