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

Proper pt_br retranslation. #675

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Proper pt_br retranslation. #675

wants to merge 2 commits into from

Conversation

ghost
Copy link

@ghost ghost commented Mar 28, 2021

Now it's an acceptable translation. The previous one had many contradictions to itself, and some phrases were just plain wrong.
Sorry if I did anything wrong, I don't anything about JavaScript. There's some comments that should be read right at the start of the file.

On line 192, I interpreted "learned to look ahead" as in "learned to look forward" since that makes more sense with the rest of the perk. Could have interpreted wrong, though, in that case I'll fix it.

Also, I don't know the reason of having a blank "" on the first line, then the phrase on the line right after. I assume it's because of spacing issues. I don't know how to test that, but wherever there was one of these cases I left it that way.

Now it's an acceptable translation. The previous one had many contradictions to itself, and some phrases were just plain wrong.
@ghost ghost changed the title Proper retranslation. Proper pt_br retranslation. Mar 28, 2021
@ghost
Copy link
Author

ghost commented Mar 28, 2021

I didn't see the translation wiki page until I was checking how often pull requests are merged. Sorry about that.

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

Successfully merging this pull request may close these issues.

0 participants