-
Notifications
You must be signed in to change notification settings - Fork 25
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
Fix wrong translations #85
Conversation
locale/drawers.he.tr
Outdated
@@ -6,7 +6,7 @@ Empty=ריקה | |||
Drawer Controller= | |||
Digiline Channel= | |||
Save= | |||
@1 (@2% full)=@1 (תפוסה של @1%) | |||
@1 (@2% full)=@1 (תפוסה של @2%) | |||
@1 @2 (@3% full)=@1 @2 (תפוסה של @3%) |
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.
@1 @2 (@3% full)=@1 @2 (תפוסה של @3%) | |
@1 @2 (@3% full)=@1 @2 (@תפוסה של 3%) |
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.
@lnjX Your change doesn't match my change:
my way: 2%@
your way: @3%
I'm not sure which way is correct in right-to-left languages
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.
Ahh i see, i thought you just missed the line. Can we test this? otherwise i'm also fine with just accepting your version.
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.
minetest support for he is broken (https://github.com/minetest/minetest/blob/7b3ed3200325ce913a6a2d884ae1ba1ccb08aad5/src/CMakeLists.txt#L668). I'm not sure I we really can test this
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.
@lnjX i18ncheck.py reports your line as wrong
Error (./drawers/locale/drawers.he.tr:10): Missing placeholder(s) in translation: '@3'
Used i18ncheck.py