Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Dutch (nl) translation improvements #418

Merged
merged 4 commits into from
Sep 25, 2018
Merged

Dutch (nl) translation improvements #418

merged 4 commits into from
Sep 25, 2018

Conversation

equaeghe
Copy link

No description provided.

Erik Quaeghebeur added 2 commits September 13, 2018 09:47
Unify nl_BE and nl_NL localization files
6bb90a0 
There was essentially no difference between these two variants.
So it would be a hassle to maintain both separately.
The difference there was (Seconden vs. Secondes) is not even
one that is a substantive difference between BE and NL. Both
are used in both regions and both are correct.
Correct and improve Dutch translations
646e5d9 
The Dutch translations contained some errors and some translations
that deviated substantially from the original. This change corrects
errors and makes some changes. The focus was on keeping the
translations close to the original and use terminology consistently.
(In case the originals are actually worse descriptions, it is those
that should be changed first, before it is propagated to the
translations.)
@equaeghe
Copy link
Author

Hmm, I just realized that also the qtpass.pro and qtpass.qrc files probably also need to be updated. I'll hold off from doing that until I get some feedback, as there may be other things I've missed.

@coveralls
Copy link

coveralls commented Sep 13, 2018 •
edited
Loading

Coverage Status

Coverage increased (+0.1%) to 6.5% when pulling e77dce9 on equaeghe:nl-vertalingverbeteringen into b4dc9e6 on IJHack:master.

@annejan
Copy link
Member

annejan commented Sep 13, 2018

I think those are the only needed changes, although I will have to see if the language loader works like this since I think it's based on language_COUNTRY (like the nl_NL, nl_BE etc).
Will probably have time to play with this tomorrow.

@equaeghe
Copy link
Author

I think those are the only needed changes,

I searched through the source files for nl_NL and only the desktop file also needed some attention.

although I will have to see if the language loader works like this since I think it's based on language_COUNTRY (like the nl_NL, nl_BE etc).

Hmm, I hope it works (should've tested it myself, sorry); for ‘ca’ (Catalan?) there is also no country code given currently.

Will probably have time to play with this tomorrow.

Ok, let me know if I need to fix something.

@annejan annejan merged commit 4fe1a73 into IJHack:master Sep 25, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@equaeghe @coveralls @annejan