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

Fixed chapter 1 contents block #35

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Fixed chapter 1 contents block #35

wants to merge 1 commit into from

Conversation

jbruni
Copy link
Contributor

@jbruni jbruni commented May 16, 2014

No description provided.

@jbruni
Copy link
Contributor Author

jbruni commented May 16, 2014

Pessoal... com a intenção de integrar a equipe, já me lanço numa primeira pequena contribuição visando a aprender melhor como contribuir (ou aprender a contribuir melhor)...

Que língua devo usar aqui? E nas mensagens de commit? Português? Inglês? Ou não tem uma "regra fixa" (depende do caso)?

Agora, sobre o "commit" específico.... bem, eu fui começar a "revisar" e logo de início me deparo com:

"Neste capítulo, você irá:

Aprenda o que faz do Meteor um framework especial.
Leia sobre a história desse livro.
Aprenda como esse livro é organizado."

Eu entendo que deve ter acontecido alguma mudança no "template" de "contents", que gerou a incompatibilidade entre o cabeçalho ("Neste capítulo você irá") e os itens ("Aprenda...", "Leia...") - seja como for, não dá para ficar assim.

Também mudei "esse" para "este" porque está referindo a si mesmo, e não a algo "distante".

De qualquer modo, a maior importância deste PR, penso eu, é eu iniciar uma integração com a equipe.

Obrigado!

@jbruni
Copy link
Contributor Author

jbruni commented May 16, 2014

Hmmm... agora que eu li um pouco mais, ao invés de modificar o "contents" dos capítulos, será que não seria o caso de mudar "Nesta capítulo, você irá:" para "Neste capítulo:" no template?

@ericdouglas
Copy link

Verdade @jbruni, o melhor a se fazer é trocar para Neste capítulo, pois todos os capítulos estão com este "problema". Vamos aguardar mais alguns colegas se manifestarem, mas a solução mais eficaz será essa mesmo!

ps: Bem vindo ao time! hehe 😄

edit: sobre a língua, creio que todos estejam usando a inglesa, mas não sei se tem alguma recomendação para que seja dessa forma.

@jbruni jbruni changed the title Fixed contents block Fixed chapter 1 contents block May 16, 2014
@lfac-pt
Copy link
Contributor

lfac-pt commented May 19, 2014

Segundo as guidelines https://www.discovermeteor.com/guidelines devemos ficar tão próximos do original em inglês quanto possível. Alterar directamente a string na UI para "Neste capítulo" iria contra essa "regra". Apesar de dar mais trabalho, eu acho que seria preferível alterar nos capitulos todos.

Em relação ao lingua, por mim é igual. A vantagem de comunicar em inglês é que se precisarmos de pedir a opinião dos autores originais do livro é mais prático, mas por mim não há problema de ser em português.

Ah, e bem vindo! 😄

@jbruni
Copy link
Contributor Author

jbruni commented May 19, 2014

Legal. Obrigado pelos Feedbacks!

@marciopamplona
Copy link
Contributor

Gostaria de contribuir, como tenho acesso a versão original ?
PS: fiz dois pulls, para sacar a idéia.

@sauveregarder
Copy link
Member

@marciopamplona, you can ask for access from @SachaG.

@SachaG
Copy link
Member

SachaG commented Jun 18, 2014

@marciopamplona just added you to the team :)

@guilhermeslk
Copy link
Contributor

Fala pessoal, também gostaria de colaborar com a tradução!

@SachaG I'd like to join the team... would you grant me access to the english sources?

@SachaG
Copy link
Member

SachaG commented Jun 28, 2014

@guilhermeslk Done :)

@guilhermeslk
Copy link
Contributor

Thanks =D

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.

7 participants