Skip to content

Arquivo index.html criado#2

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

Arquivo index.html criado#2
DanieleLimaSantos wants to merge 1 commit into
WoMakersCode:masterfrom
DanieleLimaSantos:master

Conversation

@DanieleLimaSantos

Copy link
Copy Markdown

Esta é a primeira parte do desafio: Estruturação do HTML.

Esta é a primeira parte do desafio:  Estruturação do HTML.
Comment thread index.html
<head>
<meta charset="UTF-8">
<title></title>
<link rel="stylesheet" href="https://cdnjs.cloudflare.com/ajax/libs/<font-> </font->awesome/4.7.0/css/font-awesome.min.css">

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Atenção: Acho que seu editor auto-completou erradamente uma tag alí e se meteu no meio do FontAwesome!

Comment thread index.html
</form>

<h4>Mídias</h4>
<hr>

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

@tarcilas tarcilas left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Parabéns, Daniele, bom trabalho!

Comment thread index.html
<html lang="pt-br">
<head>
<meta charset="UTF-8">
<title></title>

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Escreva um título para a página e coloque o favicon (icon.ico).

Comment thread index.html
</div>

<nav id="menu">
<a href="#">HOME</a>

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Coloque as referências para as respectivas seções.

Comment thread index.html
<div id="banner">
<img src="images/Slider.png" alt="">
<img src="images/slider01.png" alt="">
<article>

@tarcilas tarcilas Dec 13, 2018

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Acho que tag article não foi muito bem empregada, essa tag é normalmente usada para separar artigos, como em blogs, por exemplo. Eu usaria a tag section, para separar as seções definidas no menu.

Comment thread index.html
<!--Início da sessão de serviços-->
<section>
<img src="images/Service1.png" alt="">
<h3>Websites</h3>

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

É uma boa prática respeitar a hierarquia dos Headings h1, h2, h3..., não pular de h1 para h3 só por causa do tamanho da fonte, pois isso depois será mudado na aplicação de estilos.

Comment thread index.html
<hr>
</article>

<aside>

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Não usaria a tag aside neste menu, pois ele está em uma das seções principais do site, não está à parte (aside), ou seja, destacado nas laterais.

Comment thread index.html
<!--Divisão Quem sou eu-->
<article>
<h1>Quem sou eu</h1>
<p>Duis aute irure dolor in reprehenderit in voluptate velit esse cillum dolore eu fugiat nulla pariatur. Excepteur sint occaecat cupidatat non.</p>

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Você já pode customizar o texto, foto, contatos e redes sociais, afinal esse será seu portfólio.

tarcilas added a commit that referenced this pull request Dec 15, 2018
Inclusão do link para o Desafio #2
@happymoon happymoon mentioned this pull request Dec 16, 2018
pekenaerik added a commit to pekenaerik/front-end-study-group-rj that referenced this pull request Apr 26, 2019
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