#6588 closed defect (fixed)
[Patch] Improve translation
| Reported by: | Casiope | Owned by: | Casiope |
|---|---|---|---|
| Priority: | normal | 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)
Change History (9)
Changed 23 months ago by Casiope
comment:1 Changed 23 months ago by stoecker
- Owner changed from team to Casiope
- Status changed from new to needinfo
comment:2 Changed 23 months 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 23 months 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 22 months ago by stoecker
- Summary changed from [Patch] Improve translation to [Patch needs rework] Improve translation
Changed 21 months ago by simon04
comment:5 Changed 21 months ago by simon04
- Summary changed from [Patch needs rework] Improve translation to [Patch] Improve translation
Patch attached.
Just to consider: displaying "Recording: no" (instead of "none") is all right?
comment:6 Changed 21 months ago by stoecker
- Resolution set to fixed
- Status changed from needinfo to closed
In [4338/josm]:
comment:7 Changed 21 months ago by stoecker
Patch not used: Rather than confusing users I added properly I18n marker.
Nevertheless thanks for this patch and all the others :-)



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.