Skip to content

Conversation

vinicius77
Copy link
Contributor

@vinicius77 vinicius77 commented May 7, 2023

Tradução: Managing State > Reacting To Input State

PS.: 💡

  • Não traduzi valores de state, por exemplo empty, submitting, error etc nos snippets porque o sandbox perde sua funcionalidade;
  • Testei localmente ambos hyper links e sandboxes e aparentemente tudo está funcionando como deveria;
  • Expressões como Try out some challenges, Challenge Y out X , Solution, Show / Hide Solution etc vêm de outros componentes fora do escopo desta tradução;
  • Traduzi os alts das images para melhorar a a11y;
  • Intercambiei entre as expressões state e estado, dependendo do contexto da explicação;
  • Mantive reducer sem tradução.
  • Revisei o conteúdo múltiplas vezes, mas é sempre bom ter outra pessoa revisando.

Copy link
Contributor

@augustodg2 augustodg2 left a comment

Choose a reason for hiding this comment

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

Fiz alguns comentários pontuais, alguns sobre assuntos tratados no glossário. No geral, a tradução está muito boa, fácil de ler e bastante coesa. Os usos intercalados entre "estado" e "state" dependendo do contexto foram realmente necessários nessa página.

Copy link
Contributor Author

@vinicius77 vinicius77 left a comment

Choose a reason for hiding this comment

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

Reacting To Input With State

Sugestões adicionadas / corrigidas nos commits:

@jhonmike jhonmike added the needs +1 Needs another approval before merging label May 10, 2023
Copy link

@dieguesmosken dieguesmosken left a comment

Choose a reason for hiding this comment

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

O código traduzido atende aos padrões e está pronto para ser utilizado.

@jhonmike jhonmike added 🌐 translated Translation is completed for this page and removed needs +1 Needs another approval before merging labels May 31, 2023
@jhonmike
Copy link
Member

@vinicius77 você precisa atualizar sua branch com a main do repo

@jhonmike jhonmike added needs author response Changes requested needing author's input and removed 🌐 translated Translation is completed for this page labels May 31, 2023
@github-actions
Copy link

github-actions bot commented May 31, 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! 🙌

@vinicius77
Copy link
Contributor Author

@vinicius77 você precisa atualizar sua branch com a main do repo

Hei @jhonmike ! Branch atualizado com a main. ✔️

@jhonmike jhonmike added 🌐 translated Translation is completed for this page and removed needs author response Changes requested needing author's input labels Jun 10, 2023
@jhonmike jhonmike merged commit 4961d51 into reactjs:main Jun 10, 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.

4 participants