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

Correções de tradução do capítulo 04 até linha 80 #303

Merged
merged 1 commit into from
Oct 14, 2015
Merged

Correções de tradução do capítulo 04 até linha 80 #303

merged 1 commit into from
Oct 14, 2015

Conversation

Mazuh
Copy link
Contributor

@Mazuh Mazuh commented Oct 14, 2015

Falhas encontradas e corrigidas:

  • Um parágrafo do texto estava diferente do capítulo original;
  • Algumas coisas foram traduzidas sem "sensibilidade" ao novo idioma, porque algumas expressões perdem o sentido ou ficam em duplo sentido, logo é melhor serem trocadas por sinônimos;
  • Novamente o ponto da falta de sensibilidade, só que em questões gramaticais, tendo em vista uma pequena discrepância de regras entre os dois idiomas.

Falhas encontradas e corrigidas:
- Um parágrafo do texto estava diferente do capítulo original;
- Algumas coisas foram traduzidas sem "sensibilidade" ao novo idioma, porque algumas expressões perdem o sentido ou ficam em duplo sentido, logo é melhor serem trocadas por sinônimos;
- Novamente o ponto da falta de sensibilidade, só que em questões gramaticais, tendo em vista uma pequena discrepância de regras entre os dois idiomas.
@Mazuh
Copy link
Contributor Author

Mazuh commented Oct 14, 2015

ref "#215"

@Mazuh
Copy link
Contributor Author

Mazuh commented Oct 14, 2015

Não consegui identificar um padrão no momento de fazer destaques (identificando as palavras estrangeiras ou as pertencentes ao código) no texto, se é marcando em itálico ou cercando de 'aspas simples'. Posso padronizar em outro pull request.


Curiosamente, mesmo que a chamada para toUpperCase não passe nenhum argumento, a função de alguma forma têm acesso à string "Doh", cujo valor é uma propriedade. Como isso funciona exatamente é descrito no [Capítulo 6](https://github.com/braziljs/eloquente-javascript/blob/master/chapters/06-a-vida-secreta-dos-objetos.md).
Curiosamente, mesmo que a chamada para toUpperCase não passe nenhum argumento, a função de alguma forma tem acesso à string "Doh", cujo valor é uma propriedade. Como isso funciona exatamente é descrito no [Capítulo 6](https://github.com/braziljs/eloquente-javascript/blob/master/chapters/06-a-vida-secreta-dos-objetos.md).
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Mazuh coloque um link relativo aqui.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Não tô acostumado a contribuir em textos no github. Tem alguma sintaxe própria pra link relativo? Digo, eu usaria a tag 'a', mas imagino que deva haver alguma marcação nativa.

@SauloSilva
Copy link
Member

@Mazuh tivemos uma Issue na qual discutimos alguns padrões e registramos o resultado no CONTRIBUTING.
Se isto te ajudar nós avise, e sinta se a vontade em mandar as correções que achar necessário em um próximo PR.

Muito obrigado por contribuir ❤️

SauloSilva added a commit that referenced this pull request Oct 14, 2015
Correções de tradução do capítulo 04 até linha 80
@SauloSilva SauloSilva merged commit bf79fa5 into braziljs:master Oct 14, 2015
@Mazuh Mazuh deleted the patch-1 branch October 16, 2015 06:58
@Mazuh
Copy link
Contributor Author

Mazuh commented Oct 21, 2015

Just saying. Tem um bocadinho de coisa diferente da versão original.
Tô terminando de ajustar a tradução os poucos (tô lendo o original e conferindo a tradução ao mesmo tempo). Dei outro pull request lá.

@joaostein
Copy link
Contributor

@Mazuh com certeza, aos pouquinhos vamos fazendo esses PRs e melhorando o livro como um todo :)

valeu pelas contribuições, são muito importantes! 👍 😄

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.

None yet

3 participants