Skip to content

Conversation

@gabriellydeandrade
Copy link

No description provided.

@happymoon happymoon self-requested a review December 11, 2018 23:48
<body>
<nav>
<!-- TODO: modificar os tamanhos com css -->
<img src="../images/logo.png" alt="logo" height="50px"/>

Choose a reason for hiding this comment

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

Não é necessário especificar a medida dentro de atributos de imagem. Referência: https://www.w3schools.com/tags/tag_img.asp

Copy link
Author

@gabriellydeandrade gabriellydeandrade Dec 12, 2018

Choose a reason for hiding this comment

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

Eu acabei colocando pq a imagem estava muito grande no HTML, coloquei só de forma temporária mesmo por isso deixei o TODO ai em cima

Choose a reason for hiding this comment

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

Me referi à unidade de medida px (pixel), não é necessário informar as medidas nesse atributo, bastaria height="50"

<p>+55 21 66666-666</p>

<h3>Mídias</h3>
<hr/>
Copy link

@happymoon happymoon Dec 12, 2018

Choose a reason for hiding this comment

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

Semanticamente o <hr> tem a função de separar áreas que abordam conteúdos com temas diferentes e não deveria ser usado apenas por questões visuais. Referência: https://www.w3schools.com/tags/tag_hr.asp

Copy link
Author

Choose a reason for hiding this comment

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

Hm.. então essa linha é só com CSS mesmo?

Choose a reason for hiding this comment

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

Idealmente sim, não há motivo para gerar elementos extras de html se eles não tem valor semântico.

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