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 introducing jsx #47

Merged
merged 8 commits into from Feb 10, 2019

Conversation

ericp3reira
Copy link
Contributor

No description provided.

content/docs/introducing-jsx.md Outdated Show resolved Hide resolved
content/docs/introducing-jsx.md Outdated Show resolved Hide resolved
content/docs/introducing-jsx.md Outdated Show resolved Hide resolved
content/docs/introducing-jsx.md Outdated Show resolved Hide resolved
content/docs/introducing-jsx.md Outdated Show resolved Hide resolved
content/docs/introducing-jsx.md Outdated Show resolved Hide resolved
content/docs/introducing-jsx.md Outdated Show resolved Hide resolved
@fjoshuajr fjoshuajr added the needs review A pull request ready to be reviewed label Feb 9, 2019
tibuurcio
tibuurcio previously approved these changes Feb 9, 2019
@ericp3reira
Copy link
Contributor Author

Done, @tibuurcio! 😃

@tibuurcio tibuurcio mentioned this pull request Feb 9, 2019
Copy link
Member

@glaucia86 glaucia86 left a comment

Choose a reason for hiding this comment

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

@ericp3reira

Algumas alterações a serem feitas. Pedimos que faça os ajustes e logo iremos aprovar a sua contribuição.
Pedimos que, conforme for realizando os acertos que marque como 'Resolve Conversation' e nos sinalize para que possamos analisar novamente a tradução e enfim, aprovar o seu PR. Muito obrigada!

content/docs/introducing-jsx.md Outdated Show resolved Hide resolved
content/docs/introducing-jsx.md Outdated Show resolved Hide resolved

Instead of artificially separating *technologies* by putting markup and logic in separate files, React [separates *concerns*](https://en.wikipedia.org/wiki/Separation_of_concerns) with loosely coupled units called "components" that contain both. We will come back to components in a [further section](/docs/components-and-props.html), but if you're not yet comfortable putting markup in JS, [this talk](https://www.youtube.com/watch?v=x7cQ3mrcKaY) might convince you otherwise.
Ao invés de separar *tecnologias* artificialmente colocando markup e lógica em arquivos separados, o React [separa *conceitos*](https://pt.wikipedia.org/wiki/Separa%C3%A7%C3%A3o_de_conceitos) com unidades pouco acopladas chamadas "componentes" que contém ambos. Voltaremos aos componentes em
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
Ao invés de separar *tecnologias* artificialmente colocando markup e lógica em arquivos separados, o React [separa *conceitos*](https://pt.wikipedia.org/wiki/Separa%C3%A7%C3%A3o_de_conceitos) com unidades pouco acopladas chamadas "componentes" que contém ambos. Voltaremos aos componentes em
Ao invés de separar *tecnologias* artificialmente colocando markup e lógica em arquivos separados, o React [separa *conceitos*](https://pt.wikipedia.org/wiki/Separa%C3%A7%C3%A3o_de_conceitos) com unidades pouco acopladas chamadas "componentes" que contém em ambos. Voltaremos aos componentes em

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@glaucia86 Acredito que "contain both" aqui tem o sentido de "conter markup e lógica", certo? Acho que "em ambos" não encaixa, já que são os componentes que contém os dois.

Talvez alterar para "contém tanto o markup quanto a lógica"?

content/docs/introducing-jsx.md Outdated Show resolved Hide resolved
content/docs/introducing-jsx.md Outdated Show resolved Hide resolved
>
>We recommend using the ["Babel" language definition](http://babeljs.io/docs/editors) for your editor of choice so that both ES6 and JSX code is properly highlighted. This website uses the [Oceanic Next](https://labs.voronianski.com/oceanic-next-color-scheme/) color scheme which is compatible with it.
>Recomendamos o uso da [definição de linguagem "Babel"](http://babeljs.io/docs/editors) no seu editor preferido para que ambos os códigos em ES6 e JSX sejam devidamente realçados. Este website usa o esquema de cores [Oceanic Next](https://labs.voronianski.com/oceanic-next-color-scheme/) no qual é compatível o mesmo.
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
>Recomendamos o uso da [definição de linguagem "Babel"](http://babeljs.io/docs/editors) no seu editor preferido para que ambos os códigos em ES6 e JSX sejam devidamente realçados. Este website usa o esquema de cores [Oceanic Next](https://labs.voronianski.com/oceanic-next-color-scheme/) no qual é compatível o mesmo.
Recomendamos o uso da [definição de linguagem "Babel"](http://babeljs.io/docs/editors) no seu editor preferido para que ambos os códigos em ES6 e JSX sejam devidamente realçados. Este website usa o esquema de cores [Oceanic Next](https://labs.voronianski.com/oceanic-next-color-scheme/) no qual é compatível com o mesmo.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@glaucia86 Se tirarmos >, deixa de ser uma citação, não?

@glaucia86 glaucia86 added needs author response Changes requested needing author's input and removed needs review A pull request ready to be reviewed labels Feb 10, 2019
@glaucia86
Copy link
Member

@ericp3reira
Algumas alterações a serem feitas. Pedimos que faça os ajustes e logo iremos aprovar a sua contribuição.
Pedimos que, conforme for realizando os acertos que marque como 'Resolve Conversation' e nos sinalize para que possamos analisar novamente a tradução e enfim, aprovar o seu PR. Muito obrigada!

Assim que você concluir as suas mudanças, marque como 'Resolve Conversation' e altere para a label: 'changes already resolved it - translator'.
Muito obrigada pela sua contribuição! 😄

tibuurcio
tibuurcio previously approved these changes Feb 10, 2019
@ericp3reira
Copy link
Contributor Author

ericp3reira commented Feb 10, 2019

@glaucia86 Obrigado pela review! 😃
Deixei alguns comentários. Pode dar uma olhada, plz?

Ah, e não consigo mudar as labels. Não é necessário ser colaborador? 😬

@tibuurcio
Copy link
Collaborator

tibuurcio commented Feb 10, 2019

Done, @tibuurcio! 😃

São tantas notificações que eu tinha esquecido de aprovar após as alterações. Bom trabalho @ericp3reira! Acho que todos podem mudar as labels, depois tenta novamente no menu lateral direito. De qualquer forma mudei pra você 👍

@tibuurcio tibuurcio added feedback addressed and removed needs author response Changes requested needing author's input labels Feb 10, 2019
@ericp3reira
Copy link
Contributor Author

Done, @tibuurcio! 😃

São tantas notificações que eu tinha esquecido de aprovar após as alterações. Bom trabalho @ericp3reira! Acho que todos podem mudar as labels, tenta novamente no menu lateral direito >>

Obrigado, @tibuurcio! Imagino a loucura que está para verificar tudo.
Para mim não aparece a checkbox para adicionar labels, nem no menu direito. 😞

Copy link
Member

@glaucia86 glaucia86 left a comment

Choose a reason for hiding this comment

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

LGTM 👍
Queremos agradecer pela sua excelente contribuição! Muito obrigada! @ericp3reira

@glaucia86 glaucia86 merged commit ba78683 into reactjs:master Feb 10, 2019
@ericp3reira
Copy link
Contributor Author

Obrigado pela oportunidade, pessoal! Se eu puder ajudar em mais alguma coisa, só avisar.

Thanks for this chance, people! If I can be of any help, just let me know.

😄

@eduardomoroni
Copy link
Member

eduardomoroni commented Feb 10, 2019 via email

@ericp3reira
Copy link
Contributor Author

Claro, @eduardomoroni! Como posso começar?

@tibuurcio
Copy link
Collaborator

tibuurcio commented Feb 10, 2019

@ericp3reira dá uma olhada na lista de PR's abertos que contém a label "needs review". E aí só seguir a mesma ideia que fizemos aqui, dando sugestões de melhoria!

@tibuurcio tibuurcio added the 🌐 translated Translation is completed for this page label Feb 10, 2019
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.

None yet

5 participants