-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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 Polish translation #1989
Conversation
18d8b41
to
ee4e70d
Compare
@@ -33,10 +33,10 @@ | |||
"mejs.fullscreen": "Pełny ekran", | |||
|
|||
// mediaelementplayer-feature-jumpforward | |||
//"mejs.time-jump-forward": "Jump forward %1 second(s)", | |||
"mejs.time-jump-forward": "Przewiń do przodu o %1 sek.", |
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.
According to the rules of plurals, it should be 3 translations, and I only see one, can you add the other 2 please?
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.
Also, the mejs.time-skip-back
needs the other 2 translations
//"mejs.ad-skip": "Skip ad", | ||
//"mejs.ad-skip-info": "Skip in %1 second(s)", | ||
"mejs.ad-skip": "Pomiń reklamę", | ||
"mejs.ad-skip-info": "Pomiń za %1 sek.", |
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.
According to the rules of plurals, it should be 3 translations, and I only see one, can you add the other 2 please?
LGTM as long as you add the missing items I pointed out are addressed |
I've just added the missing grammar forms. |
4194a0c
to
5301ece
Compare
Thanks for your PR. Merged now |
No description provided.