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

Fix default access restrictions of highway=pedestrian, close #393. #421

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Phyks
Copy link
Member

@Phyks Phyks commented Aug 8, 2020

highway=pedestrian with no extra tag are now greyed:

https://www.cyclosm.org/#map=18/48.87808/2.38647/cyclosm
2020-08-08-165145

highway=pedestrian + bicycle=no|dismount have the same rendering as no bicycle tag:

https://www.cyclosm.org/#map=18/48.87782/2.37984/cyclosm
2020-08-08-165352

highway=pedestrian + bicycle=yes|designated have the previous rendering with color fill (and possible overloading with speed limits):

https://www.cyclosm.org/#map=19/48.86115/2.35059
2020-08-08-165714

Closes #393.

@DerDings
Copy link

DerDings commented Oct 4, 2020

I like the idea of being able to find pedestrian areas where bicycles are allowed on the map.

But I dislike the idea of using a grey fill for other pedestrian areas. Grey is used mainly for streets which cannot be accessed by cyclists, such as motorways or private ways.
Contrary to this, pedestrian areas are in reality very similar to footways, just larger. In many countries even the traffic signs for footways and pedestrian areas are (nearly) the same.

So for more consistency I think the default fill color of pedestrian areas should be as brown as footways are.
similar feature => similar colour

@Florimondable
Copy link
Member

I'm against coloring pedestrian highway in grey (like bicycle=no) by default, since this depends on local laws.

Can I close the PR ?

@DerDings
Copy link

I'm against coloring pedestrian highway in grey (like bicycle=no) by default, since this depends on local laws.

Can I close the PR ?

You're right, wiki says

"Cycling may be allowed, depending on the country or local restrictions. This can be specified with bicycle=yes or bicycle=no."

But drawing the way prominently in green by default also does assume something which cannot clearly be determined, just the other way round.

I think prominently highlighting ways as cycleable which are not suitable for cyclists in reality is worse than just not showing that cycling may be allowed in another road.

When looking at the traffic, pedestrian roads are more like a very broad footway rather than like a road. For cyclists, they're usually safe. But they're not suitable for long-way journeys, as you have to go slow. So I'd ditch the bright green entirely, and use path fill colors instead.

That would be
@path-fill green if bicycle=yes
@standard-nobicycle grey if bicycle=no
@footway-fill brown as default, if bicycle is not given

It still would look sufficiently different from regular paths and footways, as the line is broader and has a prominent grey outline.

@DerDings
Copy link

When looking at the traffic, pedestrian roads are more like a very broad footway rather than like a road. For cyclists, they're usually safe. But they're not suitable for long-way journeys, as you have to go slow. So I'd ditch the bright green entirely, and use path fill colors instead.

That would be
@path-fill green if bicycle=yes
@standard-nobicycle grey if bicycle=no
@footway-fill brown as default, if bicycle is not given

It still would look sufficiently different from regular paths and footways, as the line is broader and has a prominent grey outline.

I didn't manage to set up the development toolchain yet, but if you could hand me SVGs oft the three places given in the initial comment, I may make mock-ups to see if this conveys the information in a plain way.

@Phyks
Copy link
Member Author

Phyks commented May 23, 2021

@DerDings Here are few SVG tiles from these areas: http://pub.phyks.me/tmp/cyclosm/.

Let me know if you need anything else. Best,

@Florimondable
Copy link
Member

But drawing the way prominently in green by default also does assume something which cannot clearly be determined, just the other way round.

I think prominently highlighting ways as cycleable which are not suitable for cyclists in reality is worse than just not showing that cycling may be allowed in another road.

I don't understand, for me the way is not drawing "prominently in green" it's a green-yellowish faded. It's really different than the bright green of a no-motor-vehicle road.

When looking at the traffic, pedestrian roads are more like a very broad footway rather than like a road. For cyclists, they're usually safe. But they're not suitable for long-way journeys, as you have to go slow. So I'd ditch the bright green entirely, and use path fill colors instead.

That would be
@path-fill green if bicycle=yes
@standard-nobicycle grey if bicycle=no
@footway-fill brown as default, if bicycle is not given

It still would look sufficiently different from regular paths and footways, as the line is broader and has a prominent grey outline.

No a pedestrian highway is a road, it's a street most of the time, it's wide, you can find vehicles there.
A footway brown would emphaze the road much more.

@DerDings
Copy link

DerDings commented Jun 8, 2021

So this is how it would look like if @footway-fill was the default for pedestrian roads:

1_neu2_neu

3_neu
4_neu

6_neu
5_neu

@DerDings
Copy link

DerDings commented Jun 8, 2021

I don't understand, for me the way is not drawing "prominently in green" it's a green-yellowish faded. It's really different than the bright green of a no-motor-vehicle road.

To me it does look highlighted. And at least more similar to no-motor-vehicle roads than to other pedestrian-designated ways.
But that's the problem with individual color perception.

@Florimondable
Copy link
Member

I’m sorry but the footway color is way too strong for this kind of minor road.

If we want to change the pedestrian road color, we should go for a more faded yellowish color may be.

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.

bicycle=dismount vs bicycle=yes
3 participants