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 access= on highway=pedestrian? #4998

Open
osmuser63783 opened this issue Aug 1, 2024 · 2 comments
Open

Render access= on highway=pedestrian? #4998

osmuser63783 opened this issue Aug 1, 2024 · 2 comments
Labels

Comments

@osmuser63783
Copy link

osmuser63783 commented Aug 1, 2024

Expected behavior

Show highway=pedestrian access=no with a pattern similar to the one used for other streets with access=no

Actual behavior

No rendering of access= on highway=pedestrian. I would be curious if this is a deliberate decision - as it it seems inconsistent with the rendering of access restrictions in other highway types.

I noticed this when looking at a pedestrian bridge that had been closed and marked with access=no (it has since reopened). I was surprised to see that it was tagged access=no but did not "look closed" on the map.

Searching Overpass for examples of access=no on highway=pedestrian (4,395 cases worldwide) finds more examples of this e.g. a pedestrian street being closed (to pedestrians) due to roadworks for a couple of months. It also shows examples of verbose access tagging, e.g. highway=pedestrian access=no foot=yes bicycle=yes motor_vehicle=no, so rendering access on pedestrian without taking into account foot (i.e. solving #214) would have the side effect of showing these on the map. Not sure if that would be considered a good or a bad thing, but I wanted to flag it.

Screenshots with links illustrating the problem

Screenshot_20240802_002643_Firefox
This is a screenshot of https://www.openstreetmap.org/way/5049168 before I removed access=no (because the bridge has now reopened).

@imagico imagico added the roads label Aug 2, 2024
@imagico
Copy link
Collaborator

imagico commented Aug 2, 2024

Yes, this is one of the things we discussed as potential follow-up changes to #4952. With the current system of showing only access=* and not the transport mode specific access tags showing access=no without interpreting foot=* would often be confusing. But with #4952 this could definitely be of value.

@dch0ph
Copy link
Contributor

dch0ph commented Nov 30, 2024

The straightforward fix would be to add rendering of restricted and no access marking for highway=pedestrian based on tag interpretation for footways already added in #4952 .

The only decision would be the colour of access marking to provide sufficient contrast with the grey colour of highway=pedestrian. I would suggest simply white.

A PR could include #5024 , since the logic is the same?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants