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

Troca da fonte Roboto pela fonte Inter #322

Open
wants to merge 4 commits into
base: develop
Choose a base branch
from

Conversation

geueds
Copy link

@geueds geueds commented Jun 13, 2020

A fonte Roboto utilizada é simplesmente péssima e estraga completamente a experiência de navegar no site, principalmente à noite.

A fonte Inter é uma fonte open-source gratuita e é bem melhor para visualização.

A fonte Roboto utilizada é simplesmente péssima e estraga completamente a experiência de navegar no site, principalmente à noite.

A fonte Inter é uma fonte open-source gratuita e é bem melhor para visualização.
@papb
Copy link

papb commented Jun 13, 2020

A fonte Roboto só é pessima se você não tem o ClearType configurado corretamente no seu computador, isso já me incomodou bastante também, até que alguém finalmente me ensinou isso.

image

@geueds
Copy link
Author

geueds commented Jun 13, 2020

Provavelmente o problema seja porque a fonte Roboto não é importada no CSS e a que eu tenho instalada no meu computador é uma versão mais fina. Ainda proponho a Inter.

@papb
Copy link

papb commented Jun 13, 2020

a fonte Roboto não é importada no CSS

Ah, então realmente é um problema mesmo.

@turicas
Copy link
Owner

turicas commented Jun 13, 2020

@gabrguedes obrigado pela sugestão! Você poderia, por favor:

  • Colocar todos os arquivos necessários dentro do repositório (para servirmos os arquivos com nosso backend)
  • Colocar um print da sua tela do jeito atual e do novo jeito proposto, para verificarmos a diferença de como aparece aqui?

Obrigado! :)

@turicas
Copy link
Owner

turicas commented Jun 29, 2020

@gabrguedes você poderia fazer as alterações que comentei?

@geueds
Copy link
Author

geueds commented Jun 29, 2020

Desculpe pela demora, vou fazer daqui a pouco.

@geueds
Copy link
Author

geueds commented Jun 30, 2020

Não sei se eu fiz certo mesmo, ainda não sei muito bem como se mexe no GitHub, dentro do posssível acho que funciona.
Comparação feita com alteração de CSS em sandbox aqui (Imgur)

@@ -1,3 +1,9 @@
@import url('https://brasil.io/static/css/inter/inter.css');
Copy link
Owner

Choose a reason for hiding this comment

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

Opa, acho que aqui a URL poderia ser relativa ao endereço do CSS, não?

Copy link
Author

Choose a reason for hiding this comment

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

No caso, um /inter/inter.css apenas?

Copy link
Owner

@turicas turicas 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 atualização! Agora (provavelmente porque fiz merge de outro PR), seu branch está em conflito com o develop. Consegue ver isso?

@geueds
Copy link
Author

geueds commented Sep 16, 2020

Opa, no que acabou ficando essa request?

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