Skip to content

Desafio #1: HTML#8

Open
NayaraCN wants to merge 1 commit into
WoMakersCode:masterfrom
NayaraCN:master
Open

Desafio #1: HTML#8
NayaraCN wants to merge 1 commit into
WoMakersCode:masterfrom
NayaraCN:master

Conversation

@NayaraCN

Copy link
Copy Markdown

No description provided.

Comment thread index.html
<p>Hoje me desafio a aprender e desenvolver habilidades para ser destaque no mercado. </p>
<hr>

<figure>

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Nayara, cada imagem com sua legenda própria deverá ficar agrupada dentro de um elemento <figure> separadamente. É possível sim agrupar várias imagens dentro do figure, mas somente quando elas compartilharem da mesma legenda. Link de referência.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Suggested change
<figure>
<figure>
<img src="images/Service1.png" alt="WebSite" width="50" height="50"/>
<figcaption>
<h4>WebSites</h4>
<p>Blablablá</p>
</figcaption>
</figure>
<figure>
<img src="images/Service2.png" alt="Templates" width="50" height="50"/>
<figcaption>
<h4>Templates</h4>
<p>balabalabalabalabala</p>
</figcaption>
</figure>
<figure>
<img src="images/Service3.png" alt="Temas Wordpress" width="50" height="50"/>
<figcaption>
<h4>Temas Wordpress</h4>
<p>vamos estudar </p>
</figcaption>
</figure>

Comment thread index.html
<div id="email">nayaracn@outlook.com</div>
<p>+55 22 999587526</p>
<p>Mídias</p>
<hr>

@happymoon happymoon Dec 16, 2018

Copy link
Copy Markdown

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

Comment thread index.html

<body><!--aparece na pagina-->

<main>

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

O <main> deve ser utilizado para colocar literalmente o conteúdo principal, não devendo ele conter <header>, <footer> ou nenhum outro conteúdo que se repete ao longo do site. Referência: https://www.w3schools.com/tags/tag_main.asp

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Suggested change
<main>
<header>
</header>
<main>
</main>
<footer>
</footer>

Comment thread index.html
</ul>
</nav><!--</div>-->
</header>
<div id="home">

@happymoon happymoon Dec 16, 2018

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Nesse caso aqui você pode usar apenas o <section>, já que o <div id="home"> está sem nenhum valor semântico e apenas exercendo a função que o <section> já está executando.

Obs: isso vale para todos os outros blocos onde você usou a mesma estrutura. O <section> serve exatamente para delimitar essas áreas de conteúdo e ele pode ser estilizado via CSS da mesma forma que o <div>, então você não precisa de ambos.

@NayaraCN NayaraCN Feb 12, 2019

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Esse <id = "home"> apareceu por conta do css que ia colocar, mas realmente não consegui me desfazer dele. Ele é usado na hora que mesclo as fotos da section home.
Obs.: eu tenho dúvida sobre o que fiz ainda, tem a ver com a semântica, mas deu certo depois de quebrar um pouco a cabeça.

Comment thread index.html
<p>Teste programador </p>
</figcaption>

<figcaption>

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Como expliquei mais acima, não se deve colocar mais de uma legenda dentro de um elemento <figure> e sim separar cada grupo de foto + legenda dentro de um <figure> individual.

Comment thread index.html
<a href="images/Portfolio03.png"><img src="images/Portfolio03.png" width="100"/></a>

<figcaption>
<h4>Thumbnail label</h4>

@happymoon happymoon Dec 16, 2018

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Semanticamente não faz muito sentido utilizar um elemento de título dentro de uma legenda de imagem. As legendas são itens secundários e complementares às imagens.

Obs: isso vale para todas as outras <figcaption> com a mesma estrutura.

Comment thread index.html
<figcaption>Joana D'arc</figcaption>
</figure>

<figure>

@happymoon happymoon Dec 16, 2018

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Não considero necessário usar o <figure> para imagens que não são fotografias e que não terão legendas. Fica a seu critério, porém eu não usaria. 💃

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Neste ponto, pensei que ficaria a fotografia da "Dona do site", pois é uma apresentação na section ''sobremim''.

Comment thread index.html
<figure>
<a href="home.html"><img src="images/icon-facebook.png" alt="facebook" width="50"/></a>

<a href="home.html"><img src="images/icon-instagram.png" alt="instragram" width="50"/></a>

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

*instagram tá escrito errado no alt 🙈

Comment thread index.html
<p>Hoje me desafio a aprender e desenvolver habilidades para ser destaque no mercado. </p>

<h3>Habilidades</h3>
<!-- a barra de progresso será mostrada no css e/ou javascript -->

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Dá pra fazer também usando o elemento <progress>, dá uma conferida: https://www.w3schools.com/tags/tag_progress.asp

Comment thread index.html
<h2>Contato</h2>
<p>Hoje me desafio a aprender e desenvolver habilidades para ser destaque no mercado. </p>
<hr>
<div id="email">nayaracn@outlook.com</div>

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Você pode facilitar a vida do teu usuário colocando seu email clicável.

Suggested change
<div id="email">nayaracn@outlook.com</div>
<a href="mailto:nayaracn@outlook.com">nayaracn@outlook.com</a>

Comment thread index.html
<p>Hoje me desafio a aprender e desenvolver habilidades para ser destaque no mercado. </p>
<hr>
<div id="email">nayaracn@outlook.com</div>
<p>+55 22 999587526</p>

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Mesma coisa do email também vale para telefones.
Referência: https://www.w3schools.com/tags/tag_a.asp

Suggested change
<p>+55 22 999587526</p>
<a href="tel:+5522999587526">+55 22 999587526</a>

@happymoon

Copy link
Copy Markdown

@NayaraCN acho que seu código está muito bom, só precisa de atenção na semântica de alguns elementos. Partiu Desafio #2? abs!

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