Skip to content

Css#30

Open
NayaraCN wants to merge 2 commits into
WoMakersCode:masterfrom
NayaraCN:css
Open

Css#30
NayaraCN wants to merge 2 commits into
WoMakersCode:masterfrom
NayaraCN:css

Conversation

@NayaraCN

@NayaraCN NayaraCN commented Aug 3, 2019

Copy link
Copy Markdown

Desafio Parte 2: meu css.

Comment thread index.html
<title>Portifólio - MySite</title>
<!-- <meta charset="utf-8"/> -->

<meta name="author" content="Nayara">

@happymoon happymoon Sep 2, 2019

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, essas tags precisam ser fechadas, pra isso basta colocar barra no final, exemplo:
<meta name="application-name" content="Desenvolvimento Profissional" />

Comment thread index.html

<main>
<!--<div class="teste"></div>-->
<div id="home">

@happymoon happymoon Sep 2, 2019

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, não tem muito sentido você utilizar tanto o <div> quanto o <section>, você pode usar só o

se quiser manter essa questão semântica

Comment thread index.html
<div id="email"><a href="mailto:nayaracn@outlook.com">ncn@email.com</a></div>

<div id="phone">
<a href="tel:+5522999999999">+55 22 999999999</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.

show de bola usar o "tel:" aqui :)

@happymoon happymoon left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Oi Nayara, parabéns! Seu código está bem estruturado e limpo. Alteraria umas coisinhas aqui e alí, mas são apenas abordagens diferentes de se fazer a mesma coisa, não tem nada errado, de fato.

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