Modify

Opened 5 years ago

Closed 5 years ago

#10716 closed enhancement (fixed)

support sidewalk in default preset

Reported by: Klumbumbus Owned by: Klumbumbus
Priority: normal Milestone: 14.11
Component: Internal preset Version:
Keywords: Cc:

Description

While displaying sidewalk=* (wiki) in the MapView would result in too much clutter (see also #10057) I think adding it to the highway presets would be fine. Any opinions?

Attachments (0)

Change History (13)

comment:1 Changed 5 years ago by Don-vip

Sounds fine.

comment:2 Changed 5 years ago by Klumbumbus

Resolution: fixed
Status: newclosed

In 7707/josm:

fix #10716 - add 'sidewalk' to appropriate highway presets; add validator warning for sidewalk=*on nodes and sidewalk=yes.

comment:3 Changed 5 years ago by Klumbumbus

Milestone: 14.11

comment:4 Changed 5 years ago by skyper

The value with highest number in use is none which follows a long the deprecated footway=both/left/right/none. Not sure if it wise to through a warning about it.

comment:5 Changed 5 years ago by skyper

Do we need the values_context="sidewalk" ?

comment:6 in reply to:  5 Changed 5 years ago by Klumbumbus

Replying to skyper:

Do we need the values_context="sidewalk" ?

I thought "both" is maybe translated to german with "beide" or "beidseits" or similar depending on the context. "beide" would not fit well for sidewalk. However beside sidewalk "both" is used only for overtaking in the presets and in german it is also translated with "beidseits" there. So we could remove values_context. However I'm not sure about other languages.

comment:7 Changed 5 years ago by Klumbumbus

In 7708/josm:

see #10716 - remove values_context="sidewalk", seems unnecessary

comment:8 Changed 5 years ago by stoecker

Are you sure the context is unneeded? This will join all translations of the same strings. For some languages this may not be right.

comment:9 Changed 5 years ago by Klumbumbus

Resolution: fixed
Status: closedreopened

comment:10 Changed 5 years ago by aceman

I also vote for keeping the context, it is important and the translation here will be different than other places using left,right,both .

comment:11 Changed 5 years ago by Klumbumbus

Then we should also add the context for overtaking!?

Last edited 5 years ago by Klumbumbus (previous) (diff)

comment:12 Changed 5 years ago by aceman

Yes, you would make me happy :)

comment:13 Changed 5 years ago by Klumbumbus

Resolution: fixed
Status: reopenedclosed

In 7709/josm:

fix #10716 - add values_context="sidewalk" and values_context="overtaking" in default preset for more precise translations

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Klumbumbus.
as The resolution will be set.
The resolution will be deleted.

Add Comment


E-mail address and name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.