Skip to content

docs(pt-br): translate react-apis and createContext page #667

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

Merged
merged 10 commits into from
Apr 27, 2023

Conversation

devlulcas
Copy link
Contributor

Olá! Essa é minha segunda vez contribuindo :)

Fiz a tradução da página Built-In React APIs e createContext.

Fiquei com um pouco de dúvida se deveria alterar exemplos de código para traduzir nomes de variáveis como SomeContext.

Gostaria de uma revisão para entender o que fazer em futuras contribuições. Obrigado desde já.

@github-actions
Copy link

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! 🙌

* [`lazy`](/reference/react/lazy) lets you defer loading a component's code until it's rendered for the first time.
* [`memo`](/reference/react/memo) lets your component skip re-renders with same props. Used with [`useMemo`](/reference/react/useMemo) and [`useCallback`.](/reference/react/useCallback)
* [`startTransition`](/reference/react/startTransition) lets you mark a state update as non-urgent. Similar to [`useTransition`.](/reference/react/useTransition)
* [`createContext`](/reference/react/createContext) permite que você defina e forneça contexto aos componentes filhos. Utilizado com [`useContext`.](/reference/react/useContext)
Copy link
Contributor

Choose a reason for hiding this comment

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

não sei se gosto da tradução literal pra "contexto", porque permite que você defina e forneça contexto aos componentes filhos fica subjetivo, não faz o mesmo sentido.

talvez devemos tentar algo como: permite que você defina e forneça propriedades através de um context aos componentes filhos. Utilizado com [useContext.]

ficaria mais descritivo e traria mais proximidade ao significado inicial, sem perder a proximidade com o português

Copy link
Contributor

Choose a reason for hiding this comment

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

oi @lirbre . Essa discussão vai ser bem relevante e acho que terá diversas opiniões: no texto original, não usam Context em maiúsculo, portanto não parecem estar usando como nome próprio e sim como a palavra em seu significado comum. Nesse sentido, faria sentido traduzir...

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Concordo com o @jotaRenan. Nessa parte eu entendi o uso de "context" com o sentido mais para o lado do contexto de uma conversa

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Agora estou considerando melhor a ideia do @lirbre. Talvez realmente seja uma boa adicionar mais informação.

permite que você defina e forneça contexto aos componentes filhos através dos valores de um context.

O que acham?

Copy link
Contributor

Choose a reason for hiding this comment

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

A fim de manter a tradução mais próxima do texto original, respeitosamente mantenho minha opinião

Copy link
Member

Choose a reason for hiding this comment

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

eu gostei do jeito que esta 🤔, acho que da para entender!! oq acham podemos seguir?

Copy link
Contributor

Choose a reason for hiding this comment

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

compreendo, excelentes pontos! acho que podemos seguir sim.

Copy link
Contributor

@cauli cauli left a comment

Choose a reason for hiding this comment

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

🥇boa tradução, gostei das nuances de palavras que você escolheu.

Fiz alguns comentários, veja se é útil :)

@github-actions
Copy link

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! 🙌

@github-actions
Copy link

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! 🙌

@github-actions
Copy link

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! 🙌

1 similar comment
@github-actions
Copy link

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! 🙌

@github-actions
Copy link

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! 🙌

@github-actions
Copy link

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! 🙌

1 similar comment
@github-actions
Copy link

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 +1 Needs another approval before merging label Apr 27, 2023
@github-actions
Copy link

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 needs author response Changes requested needing author's input and removed needs +1 Needs another approval before merging labels Apr 27, 2023
@jhonmike jhonmike merged commit 9ccf401 into reactjs:main Apr 27, 2023
@jhonmike jhonmike added 🌐 translated Translation is completed for this page and removed needs author response Changes requested needing author's input labels Apr 27, 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.

6 participants