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

Translating Implementation Notes #146

Merged
merged 13 commits into from
Mar 9, 2019

Conversation

Luanf
Copy link
Contributor

@Luanf Luanf commented Mar 7, 2019

OK - Esse arquivo é longo e denso.
Detalhes de implementação do React é um tema pouco discutido então vários termos acabaram aparecendo pela primeira vez aqui, fiz um esforço para traduzir ao português a maior parte dos termos novos mesmo que cause um certo estranhamento a principio. No entanto fica aberto a discussão aqui se faz sentido remover a tradução de alguns termos e usar o original.

Um compilado de alguns termos que traduzi:

  • Stack reconciler -> Reconciliador de pilha
  • Fiber reconciler -> Reconciliador de fibra
  • Host component -> Componente hospedeiro
  • Composite component -> Componente composto
  • Lifecycle Method -> Método do ciclo de vida (acredito que esse termo deva aparecer em outros textos, mas ano esta no glossário)
  • Mount -> Tentei manipular o substantivo "Montagem" para encaixar nas frases
  • Unmount -> Mesma coisa com "Desmontagem"
  • Internal instance -> Instância interna
  • Public instance -> Instância publica

E provavelmente mais alguns... Na medida que for decidido algumas traduções eu adicionarei elas ao Glossário.

Tenho certeza que em um texto longo assim deixei passar coisas e há o que melhorar, então aprecio os reviews 😁

PS: Tem alguns commits "Apply suggestions from code review" na minha branch pois usei a própria ferramenta de suggestions pra revisar minha tradução em um "PR simulado" no meu próprio fork. Achei prático e recomendo 👍

@netlify
Copy link

netlify bot commented Mar 7, 2019

Deploy preview for pt-br-reactjs ready!

Built with commit 547a52d

https://deploy-preview-146--pt-br-reactjs.netlify.com

@jhonmike jhonmike added the needs review A pull request ready to be reviewed label Mar 7, 2019
Copy link
Member

@eduardomoroni eduardomoroni left a comment

Choose a reason for hiding this comment

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

@Luanf show de bola a tradução, fiz alguns comentarios. Tu poderia dar uma olhada e ver o que tu acha?

content/docs/implementation-notes.md Outdated Show resolved Hide resolved
content/docs/implementation-notes.md Outdated Show resolved Hide resolved
content/docs/implementation-notes.md Outdated Show resolved Hide resolved
content/docs/implementation-notes.md Outdated Show resolved Hide resolved
content/docs/implementation-notes.md Outdated Show resolved Hide resolved
content/docs/implementation-notes.md Outdated Show resolved Hide resolved
content/docs/implementation-notes.md Outdated Show resolved Hide resolved
content/docs/implementation-notes.md Outdated Show resolved Hide resolved
content/docs/implementation-notes.md Outdated Show resolved Hide resolved
content/docs/implementation-notes.md Outdated Show resolved Hide resolved
@eduardomoroni eduardomoroni added needs +1 Needs another approval before merging 💪 long page - more than 100 lines Long page and removed needs review A pull request ready to be reviewed labels Mar 7, 2019
@eduardomoroni eduardomoroni self-assigned this Mar 7, 2019
@Luanf
Copy link
Contributor Author

Luanf commented Mar 7, 2019

Obrigado pelo review @eduardomoroni !
Fiz algumas das suas correções rápidas mas ainda vou rever alguns comentários com calma.
Se mais alguém quiser opinar nas questões levantadas, fico grato

@Luanf
Copy link
Contributor Author

Luanf commented Mar 7, 2019

Trabalhei nas questões apontadas.
Infelizmente tentei um jeito diferente de alterar aqui pela pagina mas acabou gerando um pouco de poluição nas mensagens acima.
Da próxima vez faço do bom e velho jeito tradicional mesmo.
Fico no aguardo de mais comentários e revisões!

Copy link
Contributor

@lucianomlima lucianomlima left a comment

Choose a reason for hiding this comment

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

Me perdoe pelo enorme review, mas sugeri várias mudanças para deixar a leitura mais fácil.
Tem algumas correções de acentuação também.

Uma coisa que não pude colocar nesse review foi a mudança do title no início do arquivo.
Mude para Notas de Implementação para aparecer traduzido ao ser compilado para o site.
Bom trabalho!

content/docs/implementation-notes.md Outdated Show resolved Hide resolved
content/docs/implementation-notes.md Outdated Show resolved Hide resolved
content/docs/implementation-notes.md Show resolved Hide resolved
content/docs/implementation-notes.md Outdated Show resolved Hide resolved
content/docs/implementation-notes.md Outdated Show resolved Hide resolved
content/docs/implementation-notes.md Outdated Show resolved Hide resolved
content/docs/implementation-notes.md Outdated Show resolved Hide resolved
content/docs/implementation-notes.md Outdated Show resolved Hide resolved
content/docs/implementation-notes.md Outdated Show resolved Hide resolved
content/docs/implementation-notes.md Outdated Show resolved Hide resolved
@lucianomlima
Copy link
Contributor

Esse documento é enorme. Se já me deu trabalho revisar, imagine traduzir. Parabéns man! Falta pouco! 🎉

@Luanf
Copy link
Contributor Author

Luanf commented Mar 8, 2019

Opa, obrigadão pelo ótimo review 😊
Vários pontos legais que tu viu e levantou.
Corrigi vários mas falta alguns pontinhos que estarei mexendo em brevíssimo.
Deixei alguns poucos comentários para mais discussão também 👍

@lucianomlima
Copy link
Contributor

Opa, obrigadão pelo ótimo review 😊
Vários pontos legais que tu viu e levantou.
Corrigi vários mas falta alguns pontinhos que estarei mexendo em brevíssimo.
Deixei alguns poucos comentários para mais discussão também 👍

Por nada! Marca os que você já mudou depois como resolved e não esquece de traduzir o title na linha 3. Good Job!

@eduardomoroni
Copy link
Member

@Luanf
Além do titulo, falta algo ainda?
image

@eduardomoroni eduardomoroni added needs author response Changes requested needing author's input and removed needs +1 Needs another approval before merging labels Mar 9, 2019
@Luanf
Copy link
Contributor Author

Luanf commented Mar 9, 2019

Vou fazer a correção final agora! =)

@Luanf
Copy link
Contributor Author

Luanf commented Mar 9, 2019

Os últimos 2 commits devem possuir as correções pendentes. 🔎

Copy link
Member

@eduardomoroni eduardomoroni left a comment

Choose a reason for hiding this comment

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

Belissimo trabalho @Luanf, tradução complicadinha aqui.
Assim que @lucianomlima ou alguém mais aprovar a gente mergeia.

@eduardomoroni eduardomoroni added feedback addressed and removed needs author response Changes requested needing author's input labels Mar 9, 2019
Copy link
Member

@glaucia86 glaucia86 left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@glaucia86
Copy link
Member

bitmoji

@Luanf muito obrigada pela sua excelente contribuição. Estou aprovando a sua contribuição e realizando o merge para a master! Mais uma vez muito obrigada :)

@glaucia86 glaucia86 merged commit 4eeba00 into reactjs:master Mar 9, 2019
@glaucia86 glaucia86 added 🌐 translated Translation is completed for this page and removed feedback addressed 💪 long page - more than 100 lines Long page labels Mar 9, 2019
@Luanf
Copy link
Contributor Author

Luanf commented Mar 9, 2019

Obrigado pessoal! 🙏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🌐 translated Translation is completed for this page
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants