Codigo da Adail Sena #12

adailsena01 · 2018-12-19T16:07:27Z

Index da Adail Sena

happymoon · 2018-12-19T16:42:00Z

index.html

+
+		<div id="menu">
+		<nav>
+		<p style="background-color:black;"</p>


⚠️ Faltou fechar o elemento, porem a pergunta é: qual a utilidade desse parágrafo vazio?

Suggested change

<p style="background-color:black;"</p>

<p style="background-color:black;"></p>

happymoon · 2018-12-19T16:42:12Z

index.html

+		</div>
+	</div><!-- Fim do topo -->
+	<div id="banner">
+	<p style="background-color: #fb396E;"</p>


⚠️ Faltou fechar o elemento, porem a pergunta é: qual a utilidade desse parágrafo vazio?

Suggested change

<p style="background-color: #fb396E;"</p>

<p style="background-color:black;"></p>

happymoon · 2018-12-19T16:43:29Z

index.html

+	<p>MEU PORTFÓLIO</p>
+	<p>Conheça o meu trabalho...</p>
+		<img src="images/Slider.png" alt="banner de fundo com quadrados">
+		<img src="images/slider01.png" alt="borboleta" width:"400px" height="340px">


⚠️ Não é necessário especificar a unidade da medida (no caso pixels) e em atributos de imagem se utiliza sempre a formatação atributo="valor" e não com 2 pontos (:)

Suggested change

<img src="images/slider01.png" alt="borboleta" width:"400px" height="340px">

<img src="images/slider01.png" alt="borboleta" width="400" height="340" />

happymoon · 2018-12-19T16:43:42Z

index.html

+
+
+	<div id="servicos">
+	<p style="background-color:black;"></p>


⚠️ Faltou fechar o elemento, porem a pergunta é: qual a utilidade desse parágrafo vazio?

Suggested change

<p style="background-color:black;"></p>

<p style="background-color:black;"></p>

happymoon · 2018-12-19T16:45:31Z

index.html

+
+	<div id="servicos">
+	<p style="background-color:black;"></p>
+		<h2><font color="#fb396E"> Serviços</font></h2>


Se já possui um elemento, como é o caso do <h2>, não há necessidade de colocar um outro elemento dentro para estilizar, bastando utilizar o próprio <h2> como base, exemplo:

Suggested change

<h2><font color="#fb396E"> Serviços</font></h2>

<h2 style="color:fb396E;">Serviços</h2>

Lembrando que o ideal é fazer esse uso de CSS em uma folha de estilos separada e não em linha usando o atributo style. 🌶

É uma boa prática respeitar a hierarquia dos Headings h1, h2, h3..., não usar só por causa do tamanho da fonte

Suggested change

<h2><font color="#fb396E"> Serviços</font></h2>

<h1 style="color:fb396E;"> Serviços</h1>

happymoon · 2018-12-19T16:48:56Z

index.html

+	<p style="background-color:black;"></p>
+		<h2><font color="#fb396E"> Serviços</font></h2>
+		<p>Área reservada para informações sobre os meus serviços...</p>
+		<hr/>


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

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

happymoon · 2018-12-19T16:49:19Z

index.html

+	<p style="background-color: #fdad1a;"></p>
+		<h2><font color="#fb396E"> Meus Trabalhos</font></h2>
+		<p>Trabalhos realizados em vários mercados do Brasil</p>
+		<hr/>


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

happymoon · 2018-12-19T16:49:27Z

index.html

+	<p style="background-color: #fdad1a;"></p>
+		<h2><font color="#fb396E"> Quem somos</font></h2>
+		<p>Somos uma empresa que atua em todo o Brasil levando um trabalho com qualidade</p>
+		<hr/>


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

happymoon · 2018-12-19T16:49:36Z

index.html

+	<p style="background-color: #000000;"></p>
+		<h2><font color="#fb396E"> Contato</font></h2>
+		<p>Temos diversas opções para contato. Envie os seus dados para que possamos enviar a nossa proposta</p>
+		<hr/>


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

happymoon · 2018-12-19T16:50:59Z

index.html

+
+
+		<footer id="rodape">
+		<p style="background-color:black;"</p>


⚠️ Faltou fechar o elemento, porem a pergunta é: qual a utilidade desse parágrafo vazio?

Suggested change

<p style="background-color:black;"</p>

<p style="background-color:black;"></p>

tarcilas

Parabéns, você tem um grande potencial!

tarcilas · 2018-12-20T19:17:33Z

index.html

+<head>
+	<meta charset="UTF-8">
+	<title>Womarkers Code - Adail Sena</title>
+


Colocar o favicon (icon.ico)

tarcilas · 2018-12-20T19:20:51Z

index.html

+			</nav>
+		</div>
+	</div><!-- Fim do topo -->
+	<div id="banner">


É mais semântico usar a tag section para demarcar as seções ao invés de div

tarcilas · 2018-12-20T19:23:56Z

index.html

+
+			<img src="images/logo.png" alt="Logotipo do Womarkers Code" width:"200px" height="66px" >
+
+            <a href="index.html" font color="#ffffff" text-decoration: none>HOME</font></a>


O site é de página única, por isso você tem que colocar a referência para as seções, não para outros arquivos.

Suggested change

<a href="index.html" font color="#ffffff" text-decoration: none>HOME</font></a>

<a href="#banner" font color="#ffffff" text-decoration: none>HOME</font></a>

tarcilas · 2018-12-20T19:24:41Z

index.html

+		<hr/>
+
+		    <img src="images/Service1.png" alt="Websites">
+		    <h1><font color="#ec8735">Websites</font></h1>


A tag font não é mais suportada no HTML5, o estilo da fonte deve ser aplicado com CSS

tarcilas · 2018-12-20T19:26:55Z

index.html

+		<div id="meustrab">
+	<p style="background-color: #fdad1a;"></p>
+		<h2><font color="#fb396E"> Meus Trabalhos</font></h2>
+		<p>Trabalhos realizados em vários mercados do Brasil</p>


Faltou o menu dos tipos de trabalho (Tudo, Website, Template, Tema)

tarcilas · 2018-12-20T19:30:00Z

index.html

+
+	<div id="servicos">
+	<p style="background-color:black;"></p>
+		<h2><font color="#fb396E"> Serviços</font></h2>


É uma boa prática respeitar a hierarquia dos Headings h1, h2, h3..., não usar só por causa do tamanho da fonte

Suggested change

<h2><font color="#fb396E"> Serviços</font></h2>

<h1 style="color:fb396E;"> Serviços</h1>

tarcilas · 2018-12-20T19:31:36Z

index.html

+        <h2><font color="#ec8735">Sobre mim </font></h2>
+		   <p>Trabalho na área de informática desde 1992 em diversas áreasccom Website trabalho a mais de 2 anos...</p>
+		   <h2><font color="#ec8735">Habilidades </font></h2>
+


Faltou listar as habilidades

tarcilas · 2018-12-20T19:34:28Z

index.html

+		<p  align="center">Women + Tech = strong</p>
+
+
+	</footer>


A indentação não está muito legal, isso prejudica a legibilidade do código.

adailsena01 · 2018-12-20T20:39:07Z

Tarcila, vou ver as observações farei as alterações e envio novamente, pensei em colocar section depois que terminei mas a pressa em entregar acabei deixando. obrigada Adail Enviado do meu smartphone Samsung Galaxy. -------- Mensagem original --------De : Tarcila Silva <[email protected]> Data: 20/12/2018 17:39 (GMT-03:00) Para: WoMakersCode/front-end-study-group-rj <[email protected]> Cc: adailsena01 <[email protected]>, State change <[email protected]> Assunto: Re: [WoMakersCode/front-end-study-group-rj] Codigo da Adail Sena (#12) @tarcilas commented on this pull request. Parabéns, você tem um grande potencial! In index.html:

@@ -0,0 +1,115 @@

+<!DOCTYPE html> +<html lang="pt-br"> +<head> + <meta charset="UTF-8"> + <title>Womarkers Code - Adail Sena</title> + Colocar o favicon (icon.ico) In index.html:

+ <div id="menu">

+ <nav> + <p style="background-color:black;"</p> + + <img src="images/logo.png" alt="Logotipo do Womarkers Code" width:"200px" height="66px" > + + <a href="index.html" font color="#ffffff" text-decoration: none>HOME</font></a> + <a href="servicos.html" font color="#ffffff">SERVIÇOS</font></a> + <a href="portfolio.html" font color="#ffffff">PORTFOLIO</font></a> + <a href="sobremim.html" font color="#ffffff">SOBRE MIM</font></a> + <a href="contato.html" font color="#ffffff">CONTATO</font></a> + + </nav> + </div> + </div> + <div id="banner"> É mais semântico usar a tag section para demarcar as seções ao invés de div In index.html:

+<!DOCTYPE html>

+<html lang="pt-br"> +<head> + <meta charset="UTF-8"> + <title>Womarkers Code - Adail Sena</title> + +</head> +<body> + + <div id="menu"> + <nav> + <p style="background-color:black;"</p> + + <img src="images/logo.png" alt="Logotipo do Womarkers Code" width:"200px" height="66px" > + + <a href="index.html" font color="#ffffff" text-decoration: none>HOME</font></a> O site é de página única, por isso você tem que colocar a referência para as seções, não para outros arquivos. ⬇️ Suggested change - <a href="index.html" font color="#ffffff" text-decoration: none>HOME</font></a> + <a href="#banner" font color="#ffffff" text-decoration: none>HOME</font></a> In index.html:

+ <p>BEM VIND@</p>

+ <p>MEU PORTFÓLIO</p> + <p>Conheça o meu trabalho...</p> + <img src="images/Slider.png" alt="banner de fundo com quadrados"> + <img src="images/slider01.png" alt="borboleta" width:"400px" height="340px"> + </div> + + + <div id="servicos"> + <p style="background-color:black;"></p> + <h2><font color="#fb396E"> Serviços</font></h2> + <p>Área reservada para informações sobre os meus serviços...</p> + <hr/> + + <img src="images/Service1.png" alt="Websites"> + <h1><font color="#ec8735">Websites</font></h1> A tag font não é mais suportada no HTML5, o estilo da fonte deve ser aplicado com CSS In index.html:

+ <h1><font color="#ec8735">Websites</font></h1>

+ <p>Veja os Websites feito para diversos seguimentos</p> + + <img src="images/Service2.png" alt="Templates"> + <h1><font color="#ec8735">Templates</font></h1> + <p>Escolha um modelo para o seu website, podendo fazer as customizações desejadas</p> + + <img src="images/Service3.png" alt="Temas Wordpress"> + <h1><font color="#ec8735">Websites</font></h1> + <p>Aqui você verá alguns temas</p> + + </div> + <div id="meustrab"> + <p style="background-color: #fdad1a;"></p> + <h2><font color="#fb396E"> Meus Trabalhos</font></h2> + <p>Trabalhos realizados em vários mercados do Brasil</p> Faltou o menu dos tipos de trabalho (Tudo, Website, Template, Tema) In index.html:

+ </div>

+ </div> + <div id="banner"> + <p style="background-color: #fb396E;"</p> + + <p>BEM VIND@</p> + <p>MEU PORTFÓLIO</p> + <p>Conheça o meu trabalho...</p> + <img src="images/Slider.png" alt="banner de fundo com quadrados"> + <img src="images/slider01.png" alt="borboleta" width:"400px" height="340px"> + </div> + + + <div id="servicos"> + <p style="background-color:black;"></p> + <h2><font color="#fb396E"> Serviços</font></h2> É uma boa prática respeitar a hierarquia dos Headings h1, h2, h3..., não usar só por causa do tamanho da fonte ⬇️ Suggested change - <h2><font color="#fb396E"> Serviços</font></h2> + <h1 style="color:fb396E;"> Serviços</h1> In index.html:

+ <p>Imagem 3 Imagem 3 Imagem 3 </p>

+ + </div> + + <div id="quemsomos"> + <p style="background-color: #fdad1a;"></p> + <h2><font color="#fb396E"> Quem somos</font></h2> + <p>Somos uma empresa que atua em todo o Brasil levando um trabalho com qualidade</p> + <hr/> + + <img src="images/photo.png" alt="mulher"> + <h1><font color="#ec8735">Joana D'Arc</font></h1> + <h2><font color="#ec8735">Sobre mim </font></h2> + <p>Trabalho na área de informática desde 1992 em diversas áreasccom Website trabalho a mais de 2 anos...</p> + <h2><font color="#ec8735">Habilidades </font></h2> + Faltou listar as habilidades In index.html:

+ <p>E-mail: ***@***.***</p>

+ <p>Tel.: 21-99746-8667</p> + <hr/> + <h3>Mídias</h3> + + + </div> + + + + <footer id="rodape"> + <p style="background-color:black;"</p> + <p align="center">Women + Tech = strong</p> + + + </footer> A indentação não está muito legal, isso prejudica a legibilidade do código. — You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub, or mute the thread. {"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/WoMakersCode/front-end-study-group-rj","title":"WoMakersCode/front-end-study-group-rj","subtitle":"GitHub repository","main_image_url":"https://github.githubassets.com/images/email/message_cards/header.png","avatar_image_url":"https://github.githubassets.com/images/email/message_cards/avatar.png","action":{"name":"Open in GitHub","url":"https://github.com/WoMakersCode/front-end-study-group-rj"}},"updates":{"snippets":[{"icon":"PERSON","message":"@tarcilas commented on #12"}],"action":{"name":"View Pull Request","url":"#12 (review)"}}} [ { "@context": "http://schema.org", "@type": "EmailMessage", "potentialAction": { "@type": "ViewAction", "target": "#12 (review)", "url": "#12 (review)", "name": "View Pull Request" }, "description": "View this Pull Request on GitHub", "publisher": { "@type": "Organization", "name": "GitHub", "url": "https://github.com" } } ]

adicionado o CSS

adailsena01 added 2 commits December 19, 2018 12:44

Create index.html

7b04844

Update index.html

f517277

adailsena01 closed this Dec 19, 2018

adailsena01 reopened this Dec 19, 2018

happymoon reviewed Dec 19, 2018

View reviewed changes

tarcilas reviewed Dec 20, 2018

View reviewed changes

adailsena01 added 2 commits January 3, 2019 01:36

Update index.html

d3cbf72

CSS

e7d410a

adicionado o CSS

	<p style="background-color:black;"</p>
	<p style="background-color:black;"></p>

	<p style="background-color: #fb396E;"</p>
	<p style="background-color:black;"></p>

	<img src="images/slider01.png" alt="borboleta" width:"400px" height="340px">
	<img src="images/slider01.png" alt="borboleta" width="400" height="340" />

	<h2><font color="#fb396E"> Serviços</font></h2>
	<h2 style="color:fb396E;">Serviços</h2>

	<h2><font color="#fb396E"> Serviços</font></h2>
	<h1 style="color:fb396E;"> Serviços</h1>


		<img src="images/logo.png" alt="Logotipo do Womarkers Code" width:"200px" height="66px" >

		<a href="index.html" font color="#ffffff" text-decoration: none>HOME</font></a>

	<a href="index.html" font color="#ffffff" text-decoration: none>HOME</font></a>
	<a href="#banner" font color="#ffffff" text-decoration: none>HOME</font></a>

Codigo da Adail Sena #12

Are you sure you want to change the base?

Codigo da Adail Sena #12

Uh oh!

Conversation

adailsena01 commented Dec 19, 2018

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

tarcilas left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

adailsena01 commented Dec 20, 2018 via email

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants