-
Notifications
You must be signed in to change notification settings - Fork 222
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.
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
Update pl_PL app translation for 3.8.2 #2331
Update pl_PL app translation for 3.8.2 #2331
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good by scanning through the translation ;-)
@@ -716,7 +716,7 @@ | |||
<message> | |||
<location filename="../../clientdlg.cpp" line="727"/> | |||
<source>%1 Directory</source> | |||
<translation type="unfinished"></translation> | |||
<translation>%1 Serwer</translation> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just to be sure: Server is not equivalent to directory.
</message> | ||
<message> | ||
<location filename="../../serverdlg.cpp" line="127"/> | ||
<source>Recording has been switched off, either by the UI checkbox or SIGUSR2 being received</source> | ||
<translation type="unfinished"></translation> | ||
<translation>Nagrywanie zostało wyłączone przez użytkownika lub otrzymano komunikat SIGUSR2</translation> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
</message> | ||
<message> | ||
<location filename="../../serverdlg.cpp" line="148"/> | ||
<source>Language</source> | ||
<translation type="unfinished">Język</translation> | ||
<translation>Język</translation> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
<translation>Język</translation> | |
<translation>Język (Lang)</translation> |
I think @BLumia and I talked about adding (Lang) to the box to ensure people still understand it if the selected language is unknown
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't understand from where those 'unfinished' entries came. Either QtLinguist shows all done and text editor search finds none.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do you mean in @ann0see's comment above? That's just the diff. The red part shows what it looked like before your changes (= what has been removed), and the green part shows the new parts. So everything is fine. :)
@ann0see Would it make sense to solve this (lang)
thing in the code instead of asking all translators to add it? Or would you only add it for translations which don't contain *Lang*
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Probably adding it to the code would be good; however this would also show it in English then.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me, thanks a lot!
- Translator listed in the src/util.cpp, optionally add link to PR or code
- Punctuation and spacing consistent.
- Signal words consistent (“ASIO”, “Buffer”)
- App translations: No untranslated strings (grep unfinished -5 src/res/translation/translation_$TRANSLATION*.ts)
- App translations: Only a single .ts file checked in (.qm in addition is also OK)
- Installer translations: Passes tools/check-wininstaller-translations.sh
Note: Should be squash-merged. Leaving the merge to @ann0see.
Fixes #2306
Short description of changes
Context: Fixes an issue?
Does this change need documentation? What needs to be documented and how?
Status of this Pull Request
What is missing until this pull request can be merged?
Checklist