Modify

Opened 4 years ago

Last modified 4 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 richlv, 4 years ago

Summary: Add priority=forward / priority=forward in presetsAdd priority=forward / priority=backward in presets

comment:2 by skyper, 4 years ago

Component: CoreInternal preset
Keywords: priority added

Once again, a tag with *=forward/backward on nodes. Can it not only be place on ways, even if they are short?

comment:3 by richlv, 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 Don-vip, 4 years ago

Keywords: forward backward added

comment:5 by gaben, 4 months ago

So the simple version is the following:

  • resources/data/defaultpresets.xml

     
    11251125            <reference ref="highway_max" />
    11261126            <space />
    11271127            <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" />
    11281129            <preset_link preset_name="Access Restrictions" />
    11291130        </item> <!-- Road Restrictions -->
    11301131        <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 gaben, 4 months ago

Summary: Add priority=forward / priority=backward in presets[WIP patch] Add priority=forward / priority=backward in presets

Modify Ticket

Change Properties
Set your email in Preferences
Action
as new The owner will remain team.
as The resolution will be set. Next status will be 'closed'.
to The owner will be changed from team to the specified user.
Next status will be 'needinfo'. The owner will be changed from team to richlv.
as duplicate The resolution will be set to duplicate. Next status will be 'closed'. The specified ticket will be cross-referenced with this ticket.
The owner will be changed from team to anonymous. Next status will be 'assigned'.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.