Added missing translation for weapon menu #1175

Merged
merged 5 commits into from May 17, 2016

Projects

None yet

5 participants

@markusmarkusz
Contributor

Added missing translations for weapon menu (and added some spaces)

@robotboy655 robotboy655 added the TTT label May 14, 2016
@Kefta
Contributor
Kefta commented May 14, 2016

Can you check if it's missing in the other translation files?

@markusmarkusz
Contributor

Translations for +L.set_fastsw_menu and +L.set_fastswmenu_tip are missing in the other languages too.
In Traditional Chinese missing more translations.

@Kefta
Contributor
Kefta commented May 14, 2016

Do you mind giving me a full list of the missing Trad Chinese translations? I can get them all.

@markusmarkusz
Contributor

Okay. I'm going to make a list with all missing strings for all languages.
Then you and maybe someone other can fill them.

@Kefta
Contributor
Kefta commented May 14, 2016 edited

I'm also going to clean up the English file sometime because some of these strings are very grammatically broken and inconsistent. Ex. "the menu switcher menu"

I'll submit a PR to your branch with the new translations.

@markusmarkusz
Contributor

In all languages (except English, German, Portuguese and Spanish (they are in this PR)) the following strings are missing:

  • L.set_fastsw_menu
  • L.set_fastswmenu_tip

And in Russian and Trad Chinese there is either missing the following strings:

  • L.hat_retrieve

I hope that these are all. It may be that I have overlooked some. But I'm not sure.

@Kefta
Contributor
Kefta commented May 14, 2016

Thank you; this can be merged. I'll have the rest at a later date when I can contact some IRL people.

@comedinha
Contributor

Fixed portuguese strings http://pastebin.com/jrpC6h5d

@markusmarkusz
Contributor

Is this now the complete translation? Then i'll add it to the PR.

@Kefta
Contributor
Kefta commented May 16, 2016

@markusmarkusz Although he made some grammatical fixes, the main changes were making the language strings use more South American Portuguese than European, which I guess will have a wider appeal.

@markusmarkusz
Contributor

@Kefta should I merge it to the branch?

@Kefta
Contributor
Kefta commented May 16, 2016
@markusmarkusz
Contributor

I'll add it later/tomorrow.

@markusmarkusz
Contributor

Merged it.
I merged to the false branch too. :P

@svdm svdm merged commit 2598c08 into garrynewman:master May 17, 2016
@markusmarkusz markusmarkusz deleted the unknown repository branch Jul 16, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment