Project-OSRM / osrm-text-instructions

Text instructions from OSRM route responses
BSD 2-Clause "Simplified" License
86 stars 60 forks source link

Hungarian language pull request #271

Closed alexwyattdev closed 5 years ago

alexwyattdev commented 5 years ago

I would like to request the hungarian language to be included in the OSRM-text-instructions.

yuryleb commented 5 years ago

Hungarian translation on Transifex needs to complete 164 strings yet. Maybe you could fill them there and check existing strings for correctness?

alexwyattdev commented 5 years ago

Hungarian translation on Transifex needs to complete 164 strings yet. Maybe you could fill them there and check existing strings for correctness?

I will, but how can I check for correctness? I mean if I think some is correct, how can I mark it correct?

yuryleb commented 5 years ago

Just don't touch such correct string(s) or press Approve or something like this :wink:

alexwyattdev commented 5 years ago

Hungarian translation on Transifex needs to complete 164 strings yet. Maybe you could fill them there and check existing strings for correctness?

I checked the existing ones and I filled the missing ones. I do not have review rights on Transifex though, but I would say the language strings are now pullable. :)

yuryleb commented 5 years ago

I tried to merge your translation but found many missing(?) whitespaces after {way_name}, {waypoint_name}, {distance} and other macros like below:

        "continue": {
            "default": {
                "default": "Forduljon {modifier}",
                "name": "Forduljon {modifier} és maradjon a(z) {way_name}szakaszon",
                "destination": "Forduljon {modifier} a(z) {destination} felé",
                "exit": "Forduljon be {modifier} a(z) {way_name} irányába"
            },

Could you please check this more on Transifex?

And what a(z) here mean? Alternate preposition/ending depending on used macro value? Maybe Hungarian translation also needs for grammar support?

alexwyattdev commented 5 years ago

I will check those tommorrow, thanks for the info. And a(z) means that if a word starts with a vowel then the grammar requires "az", if not, then the grammar requires "a" for examle "az Alley street" or "a Melbourne way", but its not that simple because this breaks apart when it comes to codenames instead of street names for example interstate names because it would be "az M0" instead of "a M0" and I don't think its possible to implement, or if so I don't really know the way from reading the grammar support. Also for hungarian grammar it would be grammatically correct if the street names could get the proper ending for example the way_name would be "XY utca" (XY street), and when the english says: "turn left onto XY street", the hungarian would go "fordulj balra az xy utcába", so "utca" from way_name would be modified into "utcába" and we have many different way_name endings like street/road/park/circle etc. each with its own rule of ending.

alexwyattdev commented 5 years ago

I fixed the whitespaces.

yuryleb commented 5 years ago

Thank you, now it's all right with white spaces.

There is the only last issue - I suppose the u-turn modifier should start also with small letter:

            "modifier": {
                "left": "balra",
                "right": "jobbra",
                "sharp left": "élesen balra",
                "sharp right": "élesen jobbra",
                "slight left": "enyhén balra",
                "slight right": "enyhén jobbra",
                "straight": "egyenesen",
                "uturn": "Forduljon vissza"
            },
alexwyattdev commented 5 years ago

Oh sorry, fixed :)

yuryleb commented 5 years ago

@kiskunk, please review created PR with your translation.

Let's discuss possible Hungarian grammar adding after merge.

1ec5 commented 5 years ago

Fixed in #274.