-
Notifications
You must be signed in to change notification settings - Fork 312
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 Home Page #2
Conversation
gnuns
commented
Feb 8, 2019
•
edited
Loading
edited
- src/pages/index.js
- content/home/marketing
- component-based
- declarative
- learn-once-write-anywhere
- content/home/examples
- a-component-using-external-plugins
- a-simple-component
- a-stateful-component
- an-application
Please don't forget to take a look at #4 when translating!!! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
need some few changes @gnuns
since PR title says WIP I'm moving from |
@gnuns precisa de ajuda para traduzir esta seção? Se quiseres podemos dividir em pedaços menores e pedir ajuda para traduzir :) |
@eduardomoroni se precisar posso ajudar na tradução! |
Se for dividir em pedaços menores posso ajudar. |
Show de bola, vamos esperar um tempo pelo @gnuns e ai a gente ve :) |
@gnuns alguma ajuda nessa PR? Se ficar pesado para ti, podemos fazer uma força tarefa para ajudar você na tradução. cc: @eduardomoroni @WendellAdriel @cezaraugusto o que acham? Se caso ele não responder até domingo, da gente ver com ele novamente se poderá contribuir, caso contrário, atribuir para outras pessoas? |
Opa, galera. Valeu pelo apoio! Fiquei um pouco sem tempo, mas já estou de volta ao trabalho de tradução. Falta bem pouco, acredito que ainda hoje eu subo o que falta, aí qualquer dúvida/problema eu aviso vocês! |
</div> | ||
); | ||
} | ||
} | ||
|
||
ReactDOM.render( | ||
<HelloMessage name="Taylor" />, | ||
<HelloMessage name="Daniel" />, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Só para deixar um nome mais pt-BR, mesmo. :D
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Eu to num impasse em relação à tradução dos exemplos. Entendo sua intenção, mas creio que tinha alguma questão em relação ao sync com o repo original.
Eu acho que se forem só as strings é de boa, mas gostaria de mais opiniões (@glaucia86 @cezaraugusto @eduardomoroni ) ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
hahaha eu gostei da ideia, mas quanto mais o codigo original puder ser mantido, melhor. melhor deixar como o original.
nao ha nenhum impedimento forte para nao mudar mas abre precedente para outras mudancas, por exemplo alguem pode argumentar no futuro que HelloMessage
deveria ser traduzido para MensagemDeOi
"pra deixar mais pt-BR" e eu nao vou ter um argumento pra negar.
claro, se mudar pra "Cezar" eu deixo :P brincadeira
Pronto, @glaucia86 @eduardomoroni @halian-vilela! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Blz @gnuns,
Muito boa tradução cara! Fiz só algumas sugestões.
Após aplicá-las e tirarmos algumas dúvidas com outro revisor, já aprovaremos sua contribuição.
Muito obrigado!
src/pages/index.js
Outdated
@@ -51,7 +51,7 @@ class Home extends Component { | |||
return ( | |||
<Layout location={location}> | |||
<TitleAndMetaTags | |||
title="React – A JavaScript library for building user interfaces" | |||
title="React – Uma biblioteca JavaScript para criar user interfaces" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
title="React – Uma biblioteca JavaScript para criar user interfaces" | |
title="React – Uma biblioteca JavaScript para criar interfaces do usuário" |
"user interfaces" -> interfaces do usuário?
Não me parece ruim, acha que precisa deixar em inglês?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fiquei em dúvida também e resolvi deixar em inglês. Tava pensando em traduzir como "interfaces de usuário" [1]
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
concordo que "interfaces de usuário" ficaria melhor
src/pages/index.js
Outdated
@@ -134,7 +134,7 @@ class Home extends Component { | |||
fontSize: 30, | |||
}, | |||
}}> | |||
A JavaScript library for building user interfaces | |||
Uma biblioteca JavaScript para criar user interfaces |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Uma biblioteca JavaScript para criar user interfaces | |
Uma biblioteca JavaScript para criar interfaces do usuário |
Idem acima
order: 1 | ||
domid: timer-example | ||
--- | ||
|
||
In addition to taking input data (accessed via `this.props`), a component can maintain internal state data (accessed via `this.state`). When a component's state data changes, the rendered markup will be updated by re-invoking `render()`. | ||
Além de receber dados de entrada (acessados via `this.props`), um componente pode manter dados do *state* interno (acessados via `this.state`). Quando os dados do state de um componente são alterados, o código renderizado será atualizado invocando o método `render()` novamente. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ótimo! 👏
@glaucia86 essa PR foi uma das primeiras e ja esta pendente ha algum tempo. que acha da gente aceitar "as-is" e fazer as correcoes em um "follow-up"? o @halian-vilela fez isso em outra PR e eu achei bem legal |
Concordo com @cezaraugusto |
Deploy preview for pt-br-reactjs ready! Built with commit cd45f88 |
PR migrado para #113. fiz um rebase e mantive os autores dos commits depois de |
@cezaraugusto aprovei esse PR inerente ao outro PR: #125 Desculpem ausência aqui. Estava resolvendo alguns assuntos pessoais. Mas, aprovado e mergeado :) |
Co-Authored-By: Jhon Mike <[email protected]>
* Update 2019-08-15-new-react-devtools.md * Apply suggestions from code review Co-Authored-By: Júlio Campos <[email protected]> * Apply suggestions from code review #2 Co-Authored-By: Jhon Mike <[email protected]>
* Added documentation for React.createFactory * Update beta/src/content/apis/react/createFactory.md Co-authored-by: Strek <[email protected]> * minor editorial updates (#2) * Update createFactory.md * Update createFactory.md * edits Co-authored-by: Strek <[email protected]> Co-authored-by: Holly Sweeney <[email protected]> Co-authored-by: Dan Abramov <[email protected]>