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.

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

Finnish translation has vasemmall(e/a) instead of using either vasemalle or vasemmalla correctly #260

Open
mkv123 opened this issue Aug 6, 2018 · 2 comments

Comments

@mkv123
Copy link

mkv123 commented Aug 6, 2018

In Finnish there is a difference for the word "left" if you intend to keep to the left side or something is on the left side (vasemmalla) or if you are turning leftwards (vasemmalle). Same goes for "right". In the current translation, rather than differentiating between these two cases, the slightly cryptic looking vasemmall(e/a) is used.

It appears this is mainly because the English translation is using the default case with {modifier} token and the Finnish translation has done the same.

I have a fix for this where I've used "vasemmalle" for "left" and in the few cases where "vasemmalla" is correct and {modifier} is used in the "default" case, I've added the special cases for "left" and "right" with "vasemmalla".

I'm not familiar with transifex, so I'm not sure if just editing the translation files and updating the tests is the correct way to do this.

@mkv123
Copy link
Author

mkv123 commented Aug 6, 2018

Oh, just found the quite detailed CONTRIBUTING.md, I'll see if I can figure out the transifex and override stuff.

@mkv123
Copy link
Author

mkv123 commented Aug 6, 2018

Looking at the original PR #239 already mentions this problem, and the solution I came up with is being tracked at #240.

Just FYI, by setting "vasemmalle" as the default case for "left", I think only "fork" needes the special "left" and "right" cases in Finnish.

I'll leave this open, as once #240 is resolved, the Finnish translations need to be updated.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant