Skip to content

Add cycleway:lane:SIDE where cycleway:SIDE is present#1209

Closed
tordans wants to merge 1 commit intoopenstreetmap:mainfrom
tordans:cycleway_lane
Closed

Add cycleway:lane:SIDE where cycleway:SIDE is present#1209
tordans wants to merge 1 commit intoopenstreetmap:mainfrom
tordans:cycleway_lane

Conversation

@tordans
Copy link
Copy Markdown
Collaborator

@tordans tordans commented Apr 29, 2024

Test out support for https://wiki.openstreetmap.org/wiki/Key:cycleway:lane.

This is part of #1164.


This is a draft for now to experiment via Preview with different ways to model this in the existing schema.

We might want to use a directionalCombo for this. However, that would suggest to tag data for cycleway:left:lane=* when in fact there is no cycleway:left=lane. This tries to work around this by adding separate fields with re-conditions.

Another solution for this would be ideditor/schema-builder#153.

@github-actions
Copy link
Copy Markdown

🍱 You can preview the tagging presets of this pull request here.

@tordans
Copy link
Copy Markdown
Collaborator Author

tordans commented Apr 29, 2024

Next steps

  • Solve the cyclway:both=* and cycleway=* issue in the prerequisiteTag; do we need to add another field for this?
  • Probably add a readme to the folder that explains the weird setup.
  • Move from moreFields to fields so users are nudged to add the data
  • Add "undefined" option to dropdown
  • Double check highway presets (any missing?)
  • Extract into @template from highway presets
  • Update wording "Left Bike Lane Type" (because "Bike Lanes" is the directionalCombo)

@tordans tordans changed the title Test cycleway_lane tagging with separate fields Add cycleway:lane:SIDE where cycleway:SIDE is present May 1, 2024
We might want to use a directionalCombo for this. However, that would suggest to tag data for cycleway:left:lane=* when in fact there is no cycleway:left=lane. This tries to work around this by adding separate fields with re-conditions.
@matkoniecz
Copy link
Copy Markdown
Collaborator

@tordans do you plan to do those steps?

Is anyone else interested in doing those?

Or should we close this PR around second anniversary of the last activity? (or fourth or third? probably second)

@tordans
Copy link
Copy Markdown
Collaborator Author

tordans commented Dec 14, 2025

In one of the meetups we talke about this or something similar for parking where we said that the logic to only show a sub-tag (cycleway:SIDE:lane) to show up when a specific tag is given (cycleway:SIDE=lane) should be part of the logic in iD that handles the directional combo component.

I wanted to look into this. However, given that my PRs in iD are still pending on the sidewalk part (which also touches the directional combo) I will not do so until there is activity over there.

We can, however, close this draft as something that is likely not the right approach.

@tordans tordans closed this Dec 14, 2025
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