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

docs(pt-br): translate react-hooks page #708

Merged
merged 1 commit into from
Oct 30, 2023

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