Modify

Opened 8 years ago

Closed 8 years ago

Last modified 8 years ago

#6588 closed defect (fixed)

[Patch] Improve translation

Reported by: Casiope Owned by: Casiope
Priority: normal Milestone:
Component: Core Version: latest
Keywords: Cc:

Description

Display has 3 values : analog,digital and no.

These values should be : analog,digital and none.

Attachments (2)

patch.txt (1.2 KB) - added by Casiope 8 years ago.
6588v2.patch (1.1 KB) - added by simon04 8 years ago.

Download all attachments as: .zip

Change History (9)

Changed 8 years ago by Casiope

Attachment: patch.txt added

comment:1 Changed 8 years ago by stoecker

Owner: changed from team to Casiope
Status: newneedinfo

I don't think so. "none" is no standard value in OSM database. We always use "no". We could add "display_values", but we can't change "values".

P.S. Please name your patches with extension".diff" or ".patch". "patch.txt" is no good name, as it prevents syntax highlighting.

comment:2 Changed 8 years ago by Casiope

"No" has to different etymology in english : the first means "none" ("aucun" in french, "kein" in german).
http://en.wiktionary.org/wiki/no#Etymology_1

The second is the opposite of "Yes" ("Non" in french, "Nein" in german).
http://en.wiktionary.org/wiki/no#Etymology_2

Sometimes when "No" is used as a value in JOSM, the translation uses the wrong etymology.

comment:3 Changed 8 years ago by stoecker

A said. In this case we must either add display_values, which has an entry for each entry in values. Or we simply add a values_context, which allows to translate it individual.

comment:4 Changed 8 years ago by stoecker

Summary: [Patch] Improve translation[Patch needs rework] Improve translation

Changed 8 years ago by simon04

Attachment: 6588v2.patch added

comment:5 Changed 8 years ago by simon04

Summary: [Patch needs rework] Improve translation[Patch] Improve translation

Patch attached.

Just to consider: displaying "Recording: no" (instead of "none") is all right?

comment:6 Changed 8 years ago by stoecker

Resolution: fixed
Status: needinfoclosed

In [4338/josm]:

fix #6588 - i18n issue

comment:7 Changed 8 years ago by stoecker

Patch not used: Rather than confusing users I added properly I18n marker.

Nevertheless thanks for this patch and all the others :-)

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Casiope.
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.