Modify

Opened 11 years ago

Closed 10 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 by Don-vip, 11 years ago

Sounds fine.

comment:2 by Klumbumbus, 11 years ago

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 by Klumbumbus, 11 years ago

Milestone: 14.11

comment:4 by skyper, 11 years ago

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 by skyper, 11 years ago

Do we need the values_context="sidewalk" ?

in reply to:  5 comment:6 by Klumbumbus, 11 years ago

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 by Klumbumbus, 11 years ago

In 7708/josm:

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

comment:8 by stoecker, 11 years ago

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 by Klumbumbus, 11 years ago

Resolution: fixed
Status: closedreopened

comment:10 by aceman, 10 years ago

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 by Klumbumbus, 10 years ago

Then we should also add the context for overtaking!?

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

comment:12 by aceman, 10 years ago

Yes, you would make me happy :)

comment:13 by Klumbumbus, 10 years ago

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. Next status will be 'reopened'.

Add Comment


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