Skip to content

Conversation

vitoroberto
Copy link
Contributor

O que foi feito:

Ressalva

  • Seguindo o Guia de Estilo Universal decidi não traduzir "state" para "estado" e para manter o padrão na tradução também não traduzi "context" e "effects" que estavam em itálico e títulos. Acredito que desta forma a documentação ficou de fácil entendimento mas caso contrário vou ajustando a partir das decisões que virão das reviews.

@github-actions
Copy link

github-actions bot commented Aug 9, 2023

Size changes

📦 Next.js Bundle Analysis for react-dev

This analysis was generated by the Next.js Bundle Analysis action. 🤖

This PR introduced no changes to the JavaScript bundle! 🙌

@jhonmike jhonmike added the needs review A pull request ready to be reviewed label Aug 13, 2023
Copy link

@odemur odemur left a comment

Choose a reason for hiding this comment

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

Olá @vitoroberto, como vai?
Verifiquei sua tradução e está de acordo com o Guia de Estilo Universal.

Percebi que manteve os títulos dos demais Hooks em inglês.
O que acha em deixar o Performance Hooks em inglês também?

Obrigado e tenha um ótimo dia.

@vitoroberto
Copy link
Contributor Author

Olá @odemur, tudo bem! Obrigado pelo review.

"Percebi que manteve os títulos dos demais Hooks em inglês.
O que acha em deixar o Performance Hooks em inglês também?"

Essa foi uma dúvida que eu tive durante a tradução, o motivo pelo qual decidi por traduzir o título "Performance Hooks" foi porque não contém o nome do Hook nele, diferente dos outros títulos que eu decidi por deixar em inglês para manter o padrão na tradução.

Faz sentido para você ?

@odemur
Copy link

odemur commented Oct 28, 2023

Olá @vitoroberto,

Ficou claro e entendi perfeitamente.

Obrigado!

@jhonmike jhonmike added 🌐 translated Translation is completed for this page and removed needs review A pull request ready to be reviewed labels Oct 30, 2023
@jhonmike jhonmike merged commit b9c1e93 into reactjs:main Oct 30, 2023
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.

3 participants