Skip to content

Conversation

@IT-maiara-costa
Copy link

No description provided.

@alinebezzoco
Copy link
Contributor

@IT-maiara-costa, parabéns! Ficou muito bom o seu trabalho. Bacana que você usou bastante o flexbox para posicionar os elementos. Usou também a tag <progress> para criar a barra de elementos. Testei aqui e a responsividade também está ok.

padding-bottom: 2px;
}

@media screen and (max-width: 767px){
Copy link
Contributor

@alinebezzoco alinebezzoco Feb 4, 2019

Choose a reason for hiding this comment

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

Muito bom ter usado as medias queries. Vi que está usando a medida em.

Dica: em alguns casos vi que você está usando o em, px e %. Tenta usar apenas 1 medida, neste caso o em já que estamos trabalhando com responsividade.

Copy link
Author

Choose a reason for hiding this comment

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

Obrigada, trocarei pra em.

<div class="line"></div>

<address>
<a href="mailto:[email protected]">[email protected]</a> <br> <br>
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
<a href="mailto:[email protected]">[email protected]</a> <br> <br>
<a href="mailto:[email protected]">[email protected]</a>

Dica: evite o uso da tag <br> Use apenas em último caso quando precisa fazer alguma quebra de linha específica. Neste caso você pode inserir dentro do seu css a propriedade display block para a tag a. Assim a quebra de linha é feita de forma mais clean.

Copy link
Author

Choose a reason for hiding this comment

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

Entendi, vou mudar isso também.

@happymoon
Copy link

Maiara, parabéns, não vi nada de grave que precise ser pontuado. Em geral a Aline fez boas considerações.
Eu faria um ajuste aqui e ali, mas pode ser só preciosismo meu. 💃

@happymoon happymoon self-requested a review February 10, 2019 15:44
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.

3 participants