Skip to content

Conversation

@DaianeAlvesRJ
Copy link

Estrutura do site em HTML.

@DaianeAlvesRJ
Copy link
Author

@tarcilas @happymoon
Não consegui marcar revisoras no meu pull request, parece que está bloqueado. Mas gostaria, assim que possível, de um feedback de vocês.

index.html Outdated
<img src="./images/slider01.png">
</figure>
<p>
OUTRO TEXTO AQUI!

Choose a reason for hiding this comment

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

Esse texto é relacionado à imagem acima? Então pode usar o <figcaption> nesse caso.

Choose a reason for hiding this comment

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

Suggested change
OUTRO TEXTO AQUI!
<div id="primeiro-slide">
<figure>
<img src="./images/slider01.png">
<figcaption>OUTRO TEXTO AQUI!</figcaption>
</figure>

index.html Outdated
<p>Precisando tirar aquela ideia do papel? Procurando um bom profissional para desenvolver seu site? Fale
comigo!</p>
<div id="linha">
<hr />
Copy link

@happymoon happymoon Dec 16, 2018

Choose a reason for hiding this comment

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

o <hr> visualmente gera uma linha, porém semanticamente ele 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

E no caso dele ser utilizado da forma correta, não requer nenhum elemento em volta dele, já que ele pode ser estilizado via CSS, bastando referenciá-lo:

hr {atributo: valor;}

Obs: este comentário vale para todos os <hr> que você utilizou no seu código. 💁

Copy link
Author

Choose a reason for hiding this comment

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

Certo! Então posso fazer essa linha utilizando CSS?

Choose a reason for hiding this comment

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

pode e deve, basta colocar uma borda num elemento próximo, supondo que você use o parágrafo que está logo acima, basta colocar um border-bottom nele.

index.html Outdated
<figcaption>Daiane Alves</figcaption>
</figure>

<table>

Choose a reason for hiding this comment

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

Tabelas devem ser usadas somente para exibir conteúdo em formato de dados tabelados e não para estruturar código. ⚠️

index.html Outdated
<figcaption>Daiane Alves</figcaption>
</figure>

<table>

Choose a reason for hiding this comment

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

Suggested change
<table>
<ul class="redes-sociais">
<li><a href="https://www.facebook.com/DaianeAlvesRJ"><img src="./images/facebook.png" title="facebook" /></a></li>
<li><a href="https://www.instagram.com/daianealvesrj/"><img src="./images/instagram.png" title="instagram" /></a></li>
<li><a href="https://twitter.com/daianealvesrj"><img src="./images/twitter.png" title="twitter" /></a></li>
</ul>

</div>

<div id="habilidades">
<ul>

Choose a reason for hiding this comment

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

Não seria mais prático colocar o id="habilidades" direto no elemento <ul> e eliminar esse <div>? ❓

index.html Outdated
<div id="linha-menor">
<hr />
</div>
<table>

Choose a reason for hiding this comment

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

🚫 <table>

index.html Outdated
<section id="contato">
<h1>Contato</h1>
<p>Precisando de ajuda para tirar aquela ideia do papel? Vamos bater um papo?</p>
<div id="linha">

Choose a reason for hiding this comment

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

⚠️

index.html Outdated
<section id="quem-sou-eu">
<h1>Quem sou eu?</h1>
<p>Grande incentivadora de movimentos de apoio ao software livre.</p>
<div id="linha">

Choose a reason for hiding this comment

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

⚠️

Copy link
Contributor

@tarcilas tarcilas left a comment

Choose a reason for hiding this comment

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

Ótimo trabalho! Parabéns, não espera menos de você.

Co-Authored-By: DaianeAlvesRJ <[email protected]>
@DaianeAlvesRJ
Copy link
Author

Obrigada pela força e apoio, @tarcilas

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