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

Traduzir "Estado de componente" no FAQ #129

Merged
merged 6 commits into from Feb 26, 2019

Conversation

vhfmag
Copy link
Contributor

@vhfmag vhfmag commented Feb 25, 2019

Algumas decisões que eu tomei que valem ser discutidas pra manter essa página consistente com as demais:

@netlify
Copy link

netlify bot commented Feb 25, 2019

Deploy preview for pt-br-reactjs ready!

Built with commit e13f8ca

https://deploy-preview-129--pt-br-reactjs.netlify.com

Copy link
Contributor

@Luanf Luanf left a comment

Choose a reason for hiding this comment

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

Gostei da tradução!
Sugeri alguns detalhes... São pequenas coisas e de algumas não tenho certeza do que seria mais correto.

content/docs/faq-state.md Outdated Show resolved Hide resolved
content/docs/faq-state.md Outdated Show resolved Hide resolved
content/docs/faq-state.md Outdated Show resolved Hide resolved
content/docs/faq-state.md Outdated Show resolved Hide resolved
@halian-vilela
Copy link
Collaborator

Algumas decisões que eu tomei que valem ser discutidas pra manter essa página consistente com as demais:

Oi @vhfmag

event handler tem lá no glossário, temos utilizado "manipulador de evento".

re-render ainda não tem nenhuma definição consensual, acho que dependendo do contexto dá pra ser "renderizar novamente" ou algo similar.

@vhfmag
Copy link
Contributor Author

vhfmag commented Feb 25, 2019

@halian-vilela beleza! Consertei as traduções de event handler.

Copy link
Collaborator

@halian-vilela halian-vilela left a comment

Choose a reason for hiding this comment

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

Blz @vhfmag

Sugeri mais algumas mudanças, dá uma olhadinha aí e em seguidas aceitaremos sua contribuição.

Valeu!

content/docs/faq-state.md Outdated Show resolved Hide resolved
content/docs/faq-state.md Outdated Show resolved Hide resolved
content/docs/faq-state.md Show resolved Hide resolved
content/docs/faq-state.md Outdated Show resolved Hide resolved
content/docs/faq-state.md Outdated Show resolved Hide resolved
content/docs/faq-state.md Outdated Show resolved Hide resolved
content/docs/faq-state.md Outdated Show resolved Hide resolved
content/docs/faq-state.md Outdated Show resolved Hide resolved
content/docs/faq-state.md Outdated Show resolved Hide resolved
content/docs/nav.yml Outdated Show resolved Hide resolved
@halian-vilela halian-vilela added the needs author response Changes requested needing author's input label Feb 25, 2019
Co-Authored-By: vhfmag <victorhfmag@gmail.com>
@vhfmag
Copy link
Contributor Author

vhfmag commented Feb 26, 2019

@halian-vilela modificações aceitas!

@halian-vilela halian-vilela removed the needs author response Changes requested needing author's input label Feb 26, 2019
@halian-vilela halian-vilela self-assigned this Feb 26, 2019
content/docs/faq-state.md Outdated Show resolved Hide resolved
Co-Authored-By: vhfmag <victorhfmag@gmail.com>
@halian-vilela
Copy link
Collaborator

@vhfmag

LGTM 👍

Como já foi revisado pelo @Luanf, estou aprovando!

Obrigado pelas contribuições e revisão!

@halian-vilela halian-vilela merged commit 6b04f22 into reactjs:master Feb 26, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants