Skip to content

Translate Render Props - #94

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

Translate Render Props#94
eduardomoroni merged 3 commits into
reactjs:masterfrom
italotabatinga:pt-br-render-props

Conversation

@italotabatinga

@italotabatingaitalotabatinga commented Feb 13, 2019

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

@cezaraugustocezaraugusto added the needs review A pull request ready to be reviewed label Feb 13, 2019
michellocana
michellocana suggested changes Feb 13, 2019
View reviewed changes
Comment threadcontent/docs/render-props.md Outdated
Comment threadcontent/docs/render-props.md Outdated
Comment threadcontent/docs/render-props.md Outdated
Comment threadcontent/docs/render-props.md Outdated
Comment threadcontent/docs/render-props.md Outdated
@michellocana

michellocana commented Feb 13, 2019

Copy link
Copy Markdown
Contributor

Acho que os comentários dos exemplos poderiam ser traduzidos também, para ajudar no entendimento 😄

@italotabatinga

italotabatinga commented Feb 13, 2019

Copy link
Copy Markdown
ContributorAuthor

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 😄

@eduardomoronieduardomoroni 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 approved these changes Feb 14, 2019
View reviewed changes

@eduardomoronieduardomoroni 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.

Comment threadcontent/docs/render-props.md
### 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_.

@eduardomoronieduardomoroniFeb 14, 2019

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.

@italotabatingaitalotabatingaFeb 14, 2019

Copy link
Copy Markdown
ContributorAuthor

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?

Comment threadcontent/docs/render-props.md
```

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

@eduardomoronieduardomoroniFeb 14, 2019

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

eduardomoroni commented Feb 14, 2019

Copy link
Copy Markdown
Contributor

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

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

michellocana commented Feb 14, 2019

Copy link
Copy Markdown
Contributor

@eduardomoroni me parece bom, podemos dar merge 😄

@eduardomoroni

eduardomoroni commented Feb 14, 2019

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:masterFeb 14, 2019
@eduardomoroni

eduardomoroni commented Feb 14, 2019

Copy link
Copy Markdown
Contributor

Ops, parece que eu posso burlar isso.

@tesseralistesseralis mentioned this pull request Feb 14, 2019
Closed
90 tasks
michellocana
michellocana reviewed Feb 14, 2019
View reviewed changes

@michellocanamichellocana 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
@cezaraugustocezaraugusto added 🌐 translated Translation is completed for this page and removed feedback addressed labels Feb 14, 2019
devin-ai-integrationBot pushed a commit to WalissonRodrigo/pt-br.react.dev that referenced this pull request Jul 9, 2026
@italotabatinga@eduardomoroni
Translate Render Props (reactjs#94)
7d06da1
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

2 more reviewers
@michellocanamichellocanamichellocana requested changes
@eduardomoronieduardomoronieduardomoroni approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

@eduardomoronieduardomoroni

@cezaraugustocezaraugusto

Labels

🌐 translatedTranslation is completed for this page

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@italotabatinga@michellocana@eduardomoroni@cezaraugusto