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

Tradução PT-BR 2014-05-29-touristic-autism_design.markdown #76

Merged
merged 3 commits into from Oct 3, 2019
Merged

Tradução PT-BR 2014-05-29-touristic-autism_design.markdown #76

merged 3 commits into from Oct 3, 2019

Conversation

ghost
Copy link

@ghost ghost commented Oct 1, 2019

Tradução do post referenciado na issue #74 , com ajustes adicionais:

  • Adição do highlight nos snippets css
  • Os últimos dois snippets do post original não são formatados corretamente, corrigi eles para melhorar a legibilidade

@ghost ghost changed the title 74 translate design tutorial Tradução PT-BR 2014-05-29-touristic-autism_design.markdown Oct 1, 2019
Copy link
Member

@anaschwendler anaschwendler left a comment

Choose a reason for hiding this comment

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

Oi @thaisfalbo! 🎉

Muito obrigada por contribuir e arrumar os highlights, não sei se ficaram alguns faltando, mas sugeri algumas alterações, veja se elas fazem sentido, e eu volto para revisar de novo :)

_posts/2014-05-29-touristic-autism_design.markdown Outdated Show resolved Hide resolved
_posts/2014-05-29-touristic-autism_design.markdown Outdated Show resolved Hide resolved
_posts/2014-05-29-touristic-autism_design.markdown Outdated Show resolved Hide resolved
_posts/2014-05-29-touristic-autism_design.markdown Outdated Show resolved Hide resolved
_posts/2014-05-29-touristic-autism_design.markdown Outdated Show resolved Hide resolved
_posts/2014-05-29-touristic-autism_design.markdown Outdated Show resolved Hide resolved
_posts/2014-05-29-touristic-autism_design.markdown Outdated Show resolved Hide resolved
_posts/2014-05-29-touristic-autism_design.markdown Outdated Show resolved Hide resolved
_posts/2014-05-29-touristic-autism_design.markdown Outdated Show resolved Hide resolved
_posts/2014-05-29-touristic-autism_design.markdown Outdated Show resolved Hide resolved
@ghost
Copy link
Author

ghost commented Oct 2, 2019

Fiz todas as alterações porém encontrei problemas de estilo ao usar o {% highlight %}.

Como está no post original:

Captura de Tela 2019-10-02 às 12 04 55

Adicionei o {% highlight %} e ocorre os seguintes problemas: há um espaço no final do código, o parágrafo após o código não está mais alinhado no bullet point e o bullet point em si está com a fonte menor:

Captura de Tela 2019-10-02 às 12 05 54

O espaço no final do código pode ser resolvido formatando o highlight à primeira coluna do arquivo. Entretanto, o parágrafo continua não alinhado:

Captura de Tela 2019-10-02 às 12 06 16

Pelo que parece, o bug só acontece ao colocar o {% highlight %} dentro de bullet points, provavelmente ocorre dentro de outras tags markdown.

Como a intenção deste PR é a tradução e não o design, entregarei o post utilizando {% highlight %}. Porém achei importante relatar esse problema devido às issues #69 e #66

@anaschwendler
Copy link
Member

Oi @thaisfalbo!

Desculpa a demora para revisar de novo, agora já termino de revisar.
Sobre as suas considerações:

Como a intenção deste PR é a tradução e não o design, entregarei o post utilizando {% highlight %}

Super concordo com o ponto, mas eu comentei sobre o highlight mais para manter a consistência, e como você comentou que tinha arrumado em alguns outros arquivos, pensei que talvez fosse consistente manter, mas super válido.

Porém achei importante relatar esse problema devido às issues #69 e #66

De fato.

Copy link
Member

@anaschwendler anaschwendler left a comment

Choose a reason for hiding this comment

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

Pra mim fechou, tá ótimo, vou adicionar agora! Muito obrigada @thaisfalbo 🎉

Happy hacking!

@anaschwendler anaschwendler merged commit 5e14c3b into railsgirls:master Oct 3, 2019
@anaschwendler
Copy link
Member

@anaschwendler anaschwendler assigned ghost Oct 3, 2019
@ghost ghost deleted the 74-translate-design-tutorial branch October 3, 2019 13:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant