Skip to content

Crossing and Cycleway Refactoring Notes #1410

@tordans

Description

@tordans

While working on #1201 – which is now merged (but not released at this time) – I wrote down a few follow ups that I want to look into as soon as I find the time.

I did not work on them in #1201 to keep the scope on a refactoring and cleanup of what we have.

They are not all thinks that we should to … but thinks I wanted to look into. To give them a bit more visibility I move them from the PR description here so we can reference when things are being worked on …








  • crossing=no preset

    Notes

    • Make it unsearchable (at first)
    • Don't add it to the translated field-list in order to keep this focussed on the tree main types

Research how common values are

Research if traffic_calming=table is common with crossing=traffic_signals?
And is crossing:island=yes common with crossing=traffic_signals?
If not, we could remove those from the presets for node hw=crossing + crossing=traffic_signals.
Testcase https://ideditor.netlify.app/#background=Bing&disable_features=boundaries&id=n7242739087&locale=en&map=20.00/52.33904/13.65936


infrastructure on crossing presets?

Do we want the infrastructure preset on crossings as moreFields? There are cases like https://ideditor.netlify.app/#background=Bing&disable_features=boundaries&id=w1009387182&locale=en&map=20.00/52.47153/13.46059 where the crossing is on the bridge. => We can add it to the new @templates/geometry_way_more. "bridge" is used 3k times for "crossing=*" https://taginfo.openstreetmap.org/keys/crossing#combinations


name on crossing presets?

Do we want to give name more visibility on hw=path+path=crossing because path sometimes have names? Testcase https://ideditor.netlify.app/#background=Bing&disable_features=boundaries&id=w947039863&locale=en&map=20.00/52.43023/13.44348
The name is a general field which always shows when present, it does show below the explicit fields, though, see https://pr-1201--ideditor-presets-preview.netlify.app/id/dist/#background=Bing&disable_features=boundaries&id=w947039863&locale=en&map=20.00/52.43023/13.44348 at the end of the list


width on crossing presets?

  • Do we want to add width and width:source as moreTags?
  • For the hw=path we might also add bicycle:width and footway:width as moreTags

bicycle:oneway

Is there a way to add bicycle:oneway=* as kind of synonym for oneway=* on hw=path|cycleway? Both are used but only one is visible as field.


zebra

Add "addTags" for crossing=zebra to add marking zebra. Maybe also crossing_ref=zebra? Testcase https://ideditor.netlify.app/#background=Bing&disable_features=boundaries&id=n9446412638&locale=en&map=20.00/52.32519/13.62391

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions