Skip to content

Translating Implementation Notes - #146

Merged
glaucia86 merged 13 commits into
reactjs:masterfrom
Luanf:implementation-notes-pt-br
Mar 9, 2019
Merged

glaucia86 merged 13 commits into
reactjs:masterfrom
Luanf:implementation-notes-pt-br

Conversation

@Luanf

@Luanf Luanf commented Mar 7, 2019

Copy link
Copy Markdown
Contributor

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

netlify Bot commented Mar 7, 2019

Copy link
Copy Markdown

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

@eduardomoroni eduardomoroni left a comment

Copy link
Copy Markdown
Contributor

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?

Comment thread content/docs/implementation-notes.md Outdated
Comment thread content/docs/implementation-notes.md Outdated
Comment thread content/docs/implementation-notes.md Outdated
Comment thread content/docs/implementation-notes.md Outdated
Comment thread content/docs/implementation-notes.md Outdated
Comment thread content/docs/implementation-notes.md Outdated
Comment thread content/docs/implementation-notes.md Outdated
Comment thread content/docs/implementation-notes.md Outdated
Comment thread content/docs/implementation-notes.md Outdated
Comment thread content/docs/implementation-notes.md Outdated
@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
Comment thread content/docs/implementation-notes.md Outdated
Comment thread content/docs/implementation-notes.md Outdated
Co-Authored-By: Luanf <luanegidioferreira@gmail.com>
@Luanf

Luanf commented Mar 7, 2019

Copy link
Copy Markdown
Contributor Author

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

Comment thread content/docs/implementation-notes.md Outdated
Comment thread content/docs/implementation-notes.md Outdated
Comment thread content/docs/implementation-notes.md Outdated
Comment thread content/docs/implementation-notes.md Outdated
Comment thread content/docs/implementation-notes.md Outdated
Comment thread content/docs/implementation-notes.md Outdated
Co-Authored-By: Luanf <luanegidioferreira@gmail.com>
Comment thread content/docs/implementation-notes.md Outdated
Comment thread content/docs/implementation-notes.md Outdated
@Luanf

Luanf commented Mar 7, 2019

Copy link
Copy Markdown
Contributor Author

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!

@lucianomlima lucianomlima left a comment

Copy link
Copy Markdown
Contributor

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!

Comment thread content/docs/implementation-notes.md Outdated
Comment thread content/docs/implementation-notes.md Outdated
Comment thread content/docs/implementation-notes.md
Comment thread content/docs/implementation-notes.md Outdated
Comment thread content/docs/implementation-notes.md Outdated
Comment thread content/docs/implementation-notes.md Outdated
Comment thread content/docs/implementation-notes.md Outdated
Comment thread content/docs/implementation-notes.md Outdated
Comment thread content/docs/implementation-notes.md Outdated
Comment thread content/docs/implementation-notes.md Outdated
Comment thread content/docs/implementation-notes.md Outdated
@lucianomlima

Copy link
Copy Markdown
Contributor

Esse documento Γ© enorme. Se jΓ‘ me deu trabalho revisar, imagine traduzir. ParabΓ©ns man! Falta pouco! πŸŽ‰

Co-Authored-By: Luanf <luanegidioferreira@gmail.com>
@Luanf

Luanf commented Mar 8, 2019

Copy link
Copy Markdown
Contributor Author

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
Copy Markdown
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
Copy Markdown
Contributor

@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

Luanf commented Mar 9, 2019

Copy link
Copy Markdown
Contributor Author

Vou fazer a correΓ§Γ£o final agora! =)

lucianomlima and others added 2 commits March 9, 2019 08:04
@Luanf

Luanf commented Mar 9, 2019

Copy link
Copy Markdown
Contributor Author

Os ΓΊltimos 2 commits devem possuir as correΓ§Γ΅es pendentes. πŸ”Ž

@eduardomoroni eduardomoroni left a comment

Copy link
Copy Markdown
Contributor

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

@glaucia86 glaucia86 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM πŸ‘

@glaucia86

Copy link
Copy Markdown
Contributor

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

Luanf commented Mar 9, 2019

Copy link
Copy Markdown
Contributor Author

Obrigado pessoal! πŸ™

devin-ai-integration Bot pushed a commit to WalissonRodrigo/pt-br.react.dev that referenced this pull request Jul 9, 2026
* WiP - Translating Implementation Notes

* WiP - Almost finished first pass of the whole document

* WiP - Add comments and recheck grammar.

* Apply suggestions from code review

* Apply suggestions from code review [2]

* Apply suggestions from code review [3]

* Apply suggestions from code review [4]

* Apply suggestions from code review

Co-Authored-By: Luanf <luanegidioferreira@gmail.com>

* Apply suggestions from code review

Co-Authored-By: Luanf <luanegidioferreira@gmail.com>

* Apply suggestions from code review

* Apply suggestions from code review

Co-Authored-By: Luanf <luanegidioferreira@gmail.com>

* Apply suggestions from code review

Co-Authored-By: Luanf <luanegidioferreira@gmail.com>

* Translate title and a few considerations
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