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

Translate forwardRef #773

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Translate forwardRef #773

wants to merge 2 commits into from

Conversation

jgb27
Copy link

@jgb27 jgb27 commented Aug 9, 2024

Realizada a tradução da página forwardRef

issue #555

Copy link

vercel bot commented Aug 9, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
pt-br-legacy-reactjs-org ⬜️ Ignored (Inspect) Visit Preview Aug 9, 2024 2:37am

Copy link

@bdsqqq bdsqqq left a comment

Choose a reason for hiding this comment

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

Trabalho fenomenal, a tradução ficou ótima.

Apontei algumas perguntas e algumas mudanças necessárias, em maioria relacionadas a consistência ao longo da página.

Novamente, ótimo trabalho, animado pra ver a versão revisada dessa página!

@@ -4,7 +4,7 @@ title: forwardRef

<Intro>

`forwardRef` lets your component expose a DOM node to parent component with a [ref.](/learn/manipulating-the-dom-with-refs)
`forwardRef` Permitira que seu componente exponha um nó DOM para o componente pai com um [ref.](/learn/manipulating-the-dom-with-refs)
Copy link

Choose a reason for hiding this comment

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

permitira

estando no futuro do pretérito do indicativo faz referencia a uma ação tomada no passado, mas o resto da documentação usa o presente do indicativo, ex: "retorna".

Então acho que o mais correto aqui seria "Permite que seu componente..."

@@ -268,7 +267,7 @@ const FormField = forwardRef(function FormField(props, ref) {
});
```

If that `MyInput` component forwards a ref to its `<input>`, a ref to `FormField` will give you that `<input>`:
Se o componente `MyInput` encaminhar uma referência para seu `<input>`, uma referência para `FormField` fornecerá esse `<input>`:
Copy link

Choose a reason for hiding this comment

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

No glossário, 'ref' é uma das palavras que não deve ser traduzida, acredita que devemos manter a tradução nesse caso?

Copy link

Choose a reason for hiding this comment

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

abaixo, na linha 291, temos uma sentença onde ref não é traduzido:

O componente Form define uma ref

independente da escolha de traduzir ou não traduzir, devemos atualizar todas as instancias da página para manter consistência.


Instead of exposing an entire DOM node, you can expose a custom object, called an *imperative handle,* with a more constrained set of methods. To do this, you'd need to define a separate ref to hold the DOM node:
Em vez de expor um nó DOM inteiro, você pode expor um objeto personalizado, chamado de *imperative handle*, com um conjunto mais restrito de métodos. Para fazer isso, você precisaria definir uma referência separada para manter o nó DOM:
Copy link

Choose a reason for hiding this comment

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

O glossário indica que "DOM Node" deve ser traduzido para "Nó do DOM".

@@ -381,7 +380,7 @@ const MyInput = forwardRef(function MyInput(props, ref) {
});
```

Pass the `ref` you received to [`useImperativeHandle`](/reference/react/useImperativeHandle) and specify the value you want to expose to the `ref`:
Passe o `ref` que você recebeu para [`useImperativeHandle`](/reference/react/useImperativeHandle) e especifique o valor que você deseja expor ao `ref`:
Copy link

Choose a reason for hiding this comment

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

Em outras partes do página, 'ref' foi tratada no feminino, imagino que para manter consistência com 'referência'. Poderia atualizar as instancias nas quais é usado masculino?

@jhonmike jhonmike added the needs author response Changes requested needing author's input label Nov 1, 2024
@@ -131,15 +129,16 @@ function Form() {
}
```

This `Form` component [passes a ref](/reference/react/useRef#manipulating-the-dom-with-a-ref) to `MyInput`. The `MyInput` component *forwards* that ref to the `<input>` browser tag. As a result, the `Form` component can access that `<input>` DOM node and call [`focus()`](https://developer.mozilla.org/en-US/docs/Web/API/HTMLElement/focus) on it.
Este componente `Form` [passa uma ref](/reference/react/useRef#manipulating-the-dom-with-a-ref) para `MyInput`. O componente `MyInput` encaminha essa `ref` para a tag `<input>` do navegador. Como resultado, o componente Form pode acessar esse nó DOM `<input>` e chamar [`focus()`](https://developer.mozilla.org/en-US/docs/Web/API/HTMLElement/focus) nele.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
Este componente `Form` [passa uma ref](/reference/react/useRef#manipulating-the-dom-with-a-ref) para `MyInput`. O componente `MyInput` encaminha essa `ref` para a tag `<input>` do navegador. Como resultado, o componente Form pode acessar esse nó DOM `<input>` e chamar [`focus()`](https://developer.mozilla.org/en-US/docs/Web/API/HTMLElement/focus) nele.
Este componente `Form` [passa uma ref](/reference/react/useRef#manipulating-the-dom-with-a-ref) para `MyInput`. O componente `MyInput` encaminha essa `ref` para a tag `<input>` do navegador. Como resultado, o componente Form pode acessar esse nó DOM `<input>` e chamar [`focus()`](https://developer.mozilla.org/pt-BR/docs/Web/API/HTMLElement/focus) nele.


Clicking the button will call [`play()`](https://developer.mozilla.org/en-US/docs/Web/API/HTMLMediaElement/play) and [`pause()`](https://developer.mozilla.org/en-US/docs/Web/API/HTMLMediaElement/pause) on a `<video>` DOM node. The `App` component defines a ref and passes it to the `MyVideoPlayer` component. The `MyVideoPlayer` component forwards that ref to the browser `<video>` node. This lets the `App` component play and pause the `<video>`.
Clicar no botão chamará [`play()`](https://developer.mozilla.org/en-US/docs/Web/API/HTMLMediaElement/play) e [`pause()`](https://developer.mozilla.org/en-US/docs/Web/API/HTMLMediaElement/pause) em um nó DOM `<video>`. O componente `App` define uma referência e a passa para o componente `MyVideoPlayer`. O componente `MyVideoPlayer` encaminha essa referência para o nó `<video>` do navegador. Isso permite que o componente `App` reproduza e pause o `<video>`.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
Clicar no botão chamará [`play()`](https://developer.mozilla.org/en-US/docs/Web/API/HTMLMediaElement/play) e [`pause()`](https://developer.mozilla.org/en-US/docs/Web/API/HTMLMediaElement/pause) em um nó DOM `<video>`. O componente `App` define uma referência e a passa para o componente `MyVideoPlayer`. O componente `MyVideoPlayer` encaminha essa referência para o nó `<video>` do navegador. Isso permite que o componente `App` reproduza e pause o `<video>`.
Clicar no botão chamará [`play()`](https://developer.mozilla.org/pt-BR/docs/Web/API/HTMLMediaElement/play) e [`pause()`](https://developer.mozilla.org/pt-BR/docs/Web/API/HTMLMediaElement/pause) em um nó DOM `<video>`. O componente `App` define uma referência e a passa para o componente `MyVideoPlayer`. O componente `MyVideoPlayer` encaminha essa referência para o nó `<video>` do navegador. Isso permite que o componente `App` reproduza e pause o `<video>`.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs author response Changes requested needing author's input
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants