Skip to content

Translate Render Props - #94

Merged
eduardomoroni merged 3 commits into
reactjs:masterfrom
italotabatinga:pt-br-render-props
Feb 14, 2019
Merged

eduardomoroni merged 3 commits into
reactjs:masterfrom
italotabatinga:pt-br-render-props

Conversation

@italotabatinga

Copy link
Copy Markdown
Contributor

TraduΓ§Γ£o da pΓ‘gina Render Props

Vale a pena dar uma olhada

  • Cross-Cutting Concerns -> CaracterΓ­sticas Transversais (link)
  • High-Order Components -> High-Order Components
  • Close over -> esconder

@cezaraugusto cezaraugusto added the needs review A pull request ready to be reviewed label Feb 13, 2019
Comment thread content/docs/render-props.md Outdated
Comment thread content/docs/render-props.md Outdated
Comment thread content/docs/render-props.md Outdated
Comment thread content/docs/render-props.md Outdated
Comment thread content/docs/render-props.md Outdated
@michellocana

Copy link
Copy Markdown
Contributor

Acho que os comentΓ‘rios dos exemplos poderiam ser traduzidos tambΓ©m, para ajudar no entendimento πŸ˜„

@italotabatinga

Copy link
Copy Markdown
Contributor Author

Acho que os comentΓ‘rios dos exemplos poderiam ser traduzidos tambΓ©m, para ajudar no entendimento πŸ˜„

NΓ£o tinha lembrado deles 😐 Mas traduzi agora e tambΓ©m corrigi as sugestΓ΅es acima πŸ˜„

@eduardomoroni eduardomoroni added needs +1 Needs another approval before merging and removed needs review A pull request ready to be reviewed labels Feb 14, 2019

@eduardomoroni eduardomoroni left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@italotabatinga Muito obrigado.

### Tenha cuidado ao utilizar _Render Props_ num React.PureComponent {#be-careful-when-using-render-props-with-reactpurecomponent}

Using a render prop can negate the advantage that comes from using [`React.PureComponent`](/docs/react-api.html#reactpurecomponent) if you create the function inside a `render` method. This is because the shallow prop comparison will always return `false` for new props, and each `render` in this case will generate a new value for the render prop.
Usar uma _render prop_ pode anular a vantagem de utilizar [`React.PureComponent`](/docs/react-api.html#reactpurecomponent) se vocΓͺ criar uma funΓ§Γ£o dentro de um mΓ©todo `render`. Isso se deve Γ  comparaΓ§Γ£o superficial de _prop_ sempre retornar `false` para novas _props_, e, nesse caso, cada `render` vai gerar um novo valor para a _render prop_.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Esse Γ© um dos casos que eu me pergunto se nΓ£o valeria a pena colocar o termo em ingles tambΓ©m, para facilitar a googlada marota.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Qual seria o termo? Shallow Prop Comparison?

```

## Caveats {#caveats}
## Avisos {#caveats}

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Acho que poderiamos adicionar no glossario.

@eduardomoroni

Copy link
Copy Markdown
Contributor

@michellocana se estiver bom pra ti eu mergeio este:)

@eduardomoroni eduardomoroni added feedback addressed and removed needs +1 Needs another approval before merging labels Feb 14, 2019
@michellocana

Copy link
Copy Markdown
Contributor

@eduardomoroni me parece bom, podemos dar merge πŸ˜„

@eduardomoroni

Copy link
Copy Markdown
Contributor

@michellocana tu podes aprovas o PR? O merge ta bloqueado atΓ© ter 2 aproves πŸ‘

@eduardomoroni
eduardomoroni merged commit a40e202 into reactjs:master Feb 14, 2019
@eduardomoroni

Copy link
Copy Markdown
Contributor

Ops, parece que eu posso burlar isso.

@michellocana michellocana left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

SGTM πŸ˜„

@italotabatinga
italotabatinga deleted the pt-br-render-props branch February 14, 2019 14:44
@cezaraugusto cezaraugusto added 🌐 translated Translation is completed for this page and removed feedback addressed labels Feb 14, 2019
devin-ai-integration Bot pushed a commit to WalissonRodrigo/pt-br.react.dev that referenced this pull request Jul 9, 2026
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