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

Render shared cycleway+footways different from cycleways #1325

Closed
wants to merge 1 commit into from

Conversation

tvainika
Copy link

Add 'segregated' tag to .style file to allow using it in styles.

Add different rendering to non-segregated shared cycleway and footway
by alternating red and blue just like separate cycleway and footway
are rendered.

Add longer cycleway dashes to segregated cycleway and footway.

This resolves issue #1321.

Compared to my original proposal in #1321, this has only three different renderings: Current for cycleways without foot=yes/designated tag, and then two different dash patterns for segregated cases yes or no (default). In case of not segregated, rendering is like alternating footway and cycleway. In case of segregated cycleway and footway, it has a bit longer dashes for cycleway, because such segregation is usually considered more suitable for bicycling.

Add 'segregated' tag to .style file to allow using it in styles.

Add different rendering to non-segregated shared cycleway and footway
by alternating red and blue just like separate cycleway and footway
are rendered.

Add longer cycleway dashes to segregated cycleway and footway.

This resolves issue gravitystorm#1321.
@matkoniecz
Copy link
Contributor

Please, add before/after images.

@tvainika
Copy link
Author

Here is before/after images.

Let me know what you think. I actually started thinking myself if this segregated=yes case needs still polishing if you compare it to non-combined cycleway.

cycleway_before
cycleway_after

@matthijsmelissen
Copy link
Collaborator

This requires a database reload. See also #1243.

@tvainika
Copy link
Author

Let me know if you think it would be worth doing without separate case for segregated=yes tag: Combined cycleway and footway rendered by alternating blue and red dots can be done without a database reload. (Just first part of the changes made to roads.mss.) That would resolve my original issue partially.

@kocio-pl
Copy link
Collaborator

I think red-blue-red-blue pattern is not too visible, maybe it should be less interleaved (like 3xred-3xblue-3xred-3xblue or something)?

@tvainika
Copy link
Author

Current non-combined cycleway is 1×blue+3×space repeated, non-combined footway is same pattern in red.

If combined cycleway and footway uses something like 3×blue(-3×space)-3×red(-3×space) pattern, should also non-combined cycleways and footways be modified?

@matthijsmelissen matthijsmelissen added this to the 3.x - Needs upgrade to openstreetmap-carto.style milestone Feb 24, 2015
@matthijsmelissen
Copy link
Collaborator

I think the blue/red mixed combined looks too busy. In addition, I don't think rendering shared cycleways different from cycleway-only is really necessary.

All cycleway-only roads I know are either of two categories: either there is a footway immediately next to it, or pedestrians are officially not allowed to use the cycleway but everyone does so anyway. In neither case, it's not really useful for pedestrians to know that they can't use the cycleway. Is this different in Finland (or anywhere else)?

So I will reject this proposal. But thank you anyway for the effort @tvainika!

@matkoniecz
Copy link
Contributor

either there is a footway immediately next to it, or pedestrians are officially not allowed to use the cycleway but everyone does so anyway

In Poland in the second case law allows pedestrians to use cycleways.

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.

4 participants