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

New Translation: docs/working-with-images-in-markdown.md #282

Merged
merged 22 commits into from
Aug 21, 2020
Merged

New Translation: docs/working-with-images-in-markdown.md #282

merged 22 commits into from
Aug 21, 2020

Conversation

Lucas-Dalamarta
Copy link
Contributor

Qual o objetivo dessa pull request?

  • Adição de uma nova tradução
  • Correção em uma tradução existente

Qual arquivo foi traduzido/corrigido?

Foi traduzido o arquivo working-with-images-in-markdown.md

Algum comentário em relação a tradução?

Para termos como node e inline code existem termos à serem utilizados ou pode-se empregar o termo atual ?

Copy link
Contributor

@henriquepw henriquepw left a comment

Choose a reason for hiding this comment

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

Obrigado pela ajuda @Lucas-Dalamarta! 😃

Fiz algumas sugestões para melhorar a tradução, a maioria são erros ortográficos.

Sobre sua duvida não tem problema usar node e inline code o importante nesses casos é manter um padrão no texto.

docs/docs/working-with-images-in-markdown.md Outdated Show resolved Hide resolved
docs/docs/working-with-images-in-markdown.md Outdated Show resolved Hide resolved
docs/docs/working-with-images-in-markdown.md Outdated Show resolved Hide resolved
docs/docs/working-with-images-in-markdown.md Outdated Show resolved Hide resolved
docs/docs/working-with-images-in-markdown.md Outdated Show resolved Hide resolved
docs/docs/working-with-images-in-markdown.md Outdated Show resolved Hide resolved
docs/docs/working-with-images-in-markdown.md Outdated Show resolved Hide resolved
docs/docs/working-with-images-in-markdown.md Outdated Show resolved Hide resolved
docs/docs/working-with-images-in-markdown.md Outdated Show resolved Hide resolved
docs/docs/working-with-images-in-markdown.md Outdated Show resolved Hide resolved
@Lucas-Dalamarta
Copy link
Contributor Author

Muito obrigado pelas correções!

Estarei atento para evitar esses erros futuramente, e mal posso esperar para começar uma nova tradução, o processo é muito interessante e ainda vi que aprendi um pouco mais sobre o próprio Gatsby!

Copy link
Contributor

@felipefbs felipefbs left a comment

Choose a reason for hiding this comment

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

Sugeri algumas mudanças para melhorar um pouquinho o texto traduzido quanto para melhorar a tradução em si. Mas a tradução de forma geral ficou bem legal :D

docs/docs/working-with-images-in-markdown.md Outdated Show resolved Hide resolved
docs/docs/working-with-images-in-markdown.md Outdated Show resolved Hide resolved
docs/docs/working-with-images-in-markdown.md Outdated Show resolved Hide resolved
docs/docs/working-with-images-in-markdown.md Outdated Show resolved Hide resolved
docs/docs/working-with-images-in-markdown.md Outdated Show resolved Hide resolved
docs/docs/working-with-images-in-markdown.md Outdated Show resolved Hide resolved
docs/docs/working-with-images-in-markdown.md Outdated Show resolved Hide resolved
@Lucas-Dalamarta
Copy link
Contributor Author

Lucas-Dalamarta commented Jul 23, 2020

Muito obrigado pela Review!

Estou aprendendo bastante sobre como abordar determinados casos com ajuda das reviews, e na próxima tradução, estarei trazendo um texto mais condizente com o esperado.

@thayannevls
Copy link
Contributor

@Lucas-Dalamarta Para facilitar, você pode realizar as mudanças sugeridas aqui nessa pull request mesmo e nós reavaliamos. Para fazer, é só adicionar as sugestões no seu repositório e elas irão aparecer aqui pra gente

@Lucas-Dalamarta
Copy link
Contributor Author

@thayannevls Entendi, irei adicionar as atualizações!

@Lucas-Dalamarta
Copy link
Contributor Author

Adicionei todas as atualizações solicitadas, peço desculpas pelo erro de não ter aceitado-as antes.

Futuramente irei evitar esse tipo de erro! 🚀

@thayannevls
Copy link
Contributor

Se preocupa não @Lucas-Dalamarta, não foi um erro. A ideia é aprender mesmo 😄 agora é só esperar o pessoal revisar

@henriquepw henriquepw requested review from felipefbs and removed request for iaurg and jessescn August 20, 2020 14:08

Se as imagens estiverem no mesmo diretório
If your images are in the same directory as the Markdown files, sourcing and resolving the images can be done in one configuration. For example, if your Markdown pages and images are located together in a `/pages` directory, both content types will be automatically picked up by GraphQL as part of Gatsby's data layer.
Copy link
Contributor

Choose a reason for hiding this comment

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

Acho que tu deixou passar batido esse trecho XD

Copy link
Contributor Author

Choose a reason for hiding this comment

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

NOSSA! Vou dar uma conferida aqui, peço mil perdões por isso 😪

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Pelo que vi, me parece que esse trecho está traduzido na versão atual do Doc.

Posso estar enganado, por favor me corrija nesse caso. 😀

Copy link
Contributor

Choose a reason for hiding this comment

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

Oxe. Devia ta vendo o commit errado então. Mas também é super normal de acontecer esse tipo de coisa.

Copy link
Contributor

@felipefbs felipefbs left a comment

Choose a reason for hiding this comment

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

Show! Acho que já ta bem legal a tradução!

@henriquepw henriquepw merged commit 275f85d into gatsbyjs:master Aug 21, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
/docs File refering to /docs directory
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants