Opened 4 years ago
Last modified 11 months ago
#20494 new enhancement
[WIP patch] Add priority=forward / priority=backward in presets
Reported by: | richlv | Owned by: | team |
---|---|---|---|
Priority: | normal | Milestone: | |
Component: | Internal preset | Version: | |
Keywords: | priority forward backward | Cc: |
Description
Please add priority=forward and priority=backward to allow auto-completing those values in the tag dialog.
The tag seems to be non-controversial, is documented at https://wiki.openstreetmap.org/wiki/Key:priority and fairly widely used:
Tag priority=forward appears 2226 times in the planet. https://taginfo.osm.org/tags/priority=forward
Tag priority=backward appears 1874 times in the planet. https://taginfo.osm.org/tags/priority=backward
Attachments (0)
Change History (6)
comment:1 by , 4 years ago
Summary: | Add priority=forward / priority=forward in presets → Add priority=forward / priority=backward in presets |
---|
comment:2 by , 4 years ago
Component: | Core → Internal preset |
---|---|
Keywords: | priority added |
comment:3 by , 4 years ago
Oh, I didn't even notice it being suggested for usage on nodes. That might be problematic, as somebody could add it on way end node, requiring extra validity checks...
I've only applied it on ways, and wouldn't mind the preset only suggesting it on ways - that way slightly nudging towards the desired use :)
comment:4 by , 4 years ago
Keywords: | forward backward added |
---|
comment:5 by , 11 months ago
So the simple version is the following:
-
resources/data/defaultpresets.xml
1125 1125 <reference ref="highway_max" /> 1126 1126 <space /> 1127 1127 <combo key="overtaking" text="Overtaking" values="yes,both,forward,backward,no" values_context="overtaking" values_sort="false" match="key" /> 1128 <combo key="priority" text="Priority" values="forward,backward" values_context="priority" values_sort="false" /> 1128 1129 <preset_link preset_name="Access Restrictions" /> 1129 1130 </item> <!-- Road Restrictions --> 1130 1131 <item name="Access Restrictions" icon="presets/vehicle/restriction/restrictions.svg" type="node,way,closedway,multipolygon,relation" preset_name_label="true"> <!-- There are links from MPs and route relations to this preset -->
But as you mentioned, it only makes sense on ways. Not sure how to do that, or if it's even possible with the current preset structure.
comment:6 by , 11 months ago
Summary: | Add priority=forward / priority=backward in presets → [WIP patch] Add priority=forward / priority=backward in presets |
---|
Once again, a tag with
*=forward/backward
on nodes. Can it not only be place on ways, even if they are short?