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

Estilos para o member get member do portal do consumidor #36

Merged
merged 5 commits into from
Mar 18, 2020

Conversation

CarlosMouraJR
Copy link

No description provided.

Comment on lines +261 to +264
p {
font-size: 22px;
color: white;
}

Choose a reason for hiding this comment

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

não seria melhor usar a tag h4, que se não me engano tem 22px de font-size, e também mudar a cor para #FFF

Copy link
Author

Choose a reason for hiding this comment

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

Os H no shards usam a medida rem.
Eu usei os H no começo mas mudei por dois motivos:
1 - Por ser rem, não estava entregando algo fiel a SPEC (tava me incomodando)
2 - Mobile e desktop tem tamanhos diferentes (por ex: mobile 24px, desktop 20px)

Temos duas opções:
1 - Usar os H em prol do SEO, e alterar o tamanho dele para desktop para um rem compativel ao tamanho da fonte em desktop semelhante a SPEC
2 - Manter como P com os tamanhos que defini.

text-align: left;

.description {
font-size: 26px;
Copy link

@natankruger natankruger Mar 18, 2020

Choose a reason for hiding this comment

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

acho que o h3 é o equivalente de 26px

Copy link
Author

Choose a reason for hiding this comment

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

Os H no shards usam a medida rem.
Eu usei os H no começo mas mudei por dois motivos:
1 - Por ser rem, não estava entregando algo fiel a SPEC (tava me incomodando)
2 - Mobile e desktop tem tamanhos diferentes (por ex: mobile 24px, desktop 20px)

Temos duas opções:
1 - Usar os H em prol do SEO, e alterar o tamanho dele para desktop para um rem compativel ao tamanho da fonte em desktop semelhante a SPEC
2 - Manter como P com os tamanhos que defini.

Comment on lines +326 to +329
p {
font-size: 26px;
margin: 0;
}

Choose a reason for hiding this comment

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

h3 ao invés de trocar o tamanho do p

Copy link
Author

Choose a reason for hiding this comment

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

Os H no shards usam a medida rem.
Eu usei os H no começo mas mudei por dois motivos:
1 - Por ser rem, não estava entregando algo fiel a SPEC (tava me incomodando)
2 - Mobile e desktop tem tamanhos diferentes (por ex: mobile 24px, desktop 20px)

Temos duas opções:
1 - Usar os H em prol do SEO, e alterar o tamanho dele para desktop para um rem compativel ao tamanho da fonte em desktop semelhante a SPEC
2 - Manter como P com os tamanhos que defini.

Copy link

@natankruger natankruger left a comment

Choose a reason for hiding this comment

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

Ver os pontos em que comentei, principalmente os que eu falo sobre alterar o tamanho da tag p, que isso afeta o nosso SEO, e no whitelabel o SEO é bem importante.

@CarlosMouraJR
Copy link
Author

Ver os pontos em que comentei, principalmente os que eu falo sobre alterar o tamanho da tag p, que isso afeta o nosso SEO, e no whitelabel o SEO é bem importante.

Respondi a questão sobre os H, segue a resposta:

Os H no shards usam a medida rem.
Eu usei os H no começo mas mudei por dois motivos:
1 - Por ser rem, não estava entregando algo fiel a SPEC (tava me incomodando)
2 - Mobile e desktop tem tamanhos diferentes (por ex: mobile 24px, desktop 20px)

Temos duas opções:
1 - Usar os H em prol do SEO, e alterar o tamanho dele para desktop para um rem compativel ao tamanho da fonte em desktop semelhante a SPEC
2 - Manter como P com os tamanhos que defini.

@natankruger
Copy link

Ver os pontos em que comentei, principalmente os que eu falo sobre alterar o tamanho da tag p, que isso afeta o nosso SEO, e no whitelabel o SEO é bem importante.

Respondi a questão sobre os H, segue a resposta:

Os H no shards usam a medida rem.
Eu usei os H no começo mas mudei por dois motivos:
1 - Por ser rem, não estava entregando algo fiel a SPEC (tava me incomodando)
2 - Mobile e desktop tem tamanhos diferentes (por ex: mobile 24px, desktop 20px)

Temos duas opções:
1 - Usar os H em prol do SEO, e alterar o tamanho dele para desktop para um rem compativel ao tamanho da fonte em desktop semelhante a SPEC
2 - Manter como P com os tamanhos que defini.

Tem uma terceira opção, a medida em rem tem um equivalente de px se vc dar uma olhada na aba computed do inspetor de css da pra ver qual é a equivalência, e usar uma combinação de d-flex d-md-none (não lembro de cabeça certo), com cada h equivalente para mobile e desktop.
E ainda uma quarta que é falar com o designer da tela (@jevel9 ) para ele deixar os tamanhos das fontes iguais tanto no mobile quanto no desktop.

@CarlosMouraJR
Copy link
Author

Ver os pontos em que comentei, principalmente os que eu falo sobre alterar o tamanho da tag p, que isso afeta o nosso SEO, e no whitelabel o SEO é bem importante.

Respondi a questão sobre os H, segue a resposta:
Os H no shards usam a medida rem.
Eu usei os H no começo mas mudei por dois motivos:
1 - Por ser rem, não estava entregando algo fiel a SPEC (tava me incomodando)
2 - Mobile e desktop tem tamanhos diferentes (por ex: mobile 24px, desktop 20px)
Temos duas opções:
1 - Usar os H em prol do SEO, e alterar o tamanho dele para desktop para um rem compativel ao tamanho da fonte em desktop semelhante a SPEC
2 - Manter como P com os tamanhos que defini.

Tem uma terceira opção, a medida em rem tem um equivalente de px se vc dar uma olhada na aba computed do inspetor de css da pra ver qual é a equivalência, e usar uma combinação de d-flex d-md-none (não lembro de cabeça certo), com cada h equivalente para mobile e desktop.
E ainda uma quarta que é falar com o designer da tela (@jevel9 ) para ele deixar os tamanhos das fontes iguais tanto no mobile quanto no desktop.

Se ta andando muito com o @evsasse
Olha essa gambiarra mermao.. esse q esquema de d-block d-md-none e etc, normalmente uso em último caso, e normalmente para um elemento especifico (botão e etc).

A quarta opção não axo viavel agora, o Design ta bem atarefado com o delivery do bem.

Vou mergear agora, e podemos rever isso no futuro!

@CarlosMouraJR CarlosMouraJR merged commit 9f55bc2 into master Mar 18, 2020
@CarlosMouraJR CarlosMouraJR deleted the mgm/disabled branch March 18, 2020 13:53
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.

2 participants