Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

PR de correção #13

Open
wants to merge 31 commits into
base: correcao-projeto
Choose a base branch
from
Open

PR de correção #13

wants to merge 31 commits into from

Conversation

jvpalves
Copy link
Contributor

@jvpalves jvpalves commented Mar 5, 2020

POR FAVOR, EDITE ESSA MENSAGEM INSERINDO AS INFORMAÇÕES DO SEU PROJETO. COMECE APAGANDO ESSA LINHA.

INSIRA O NOME NOME DO PROJETO AQUI

O que funciona

  • DESCREVA O QUE FUNCIONA NO SEU PROJETO

O que não funciona

  • DESCREVA O QUE NÃO FUNCIONA NO SEU PROJETO

Link Surge

A PARTIR DA SEMANA 5, INSIRA AQUI O LINK DO SURGE. ANTES DISSO, APAGUE ESSAS DUAS LINHAS.

Imagens

TIRE PRINTS DAS TELAS DO SEU SITE E COLE AQUI

jvpalves and others added 30 commits February 20, 2020 21:58
pagina equipe CSS html e arquivos
equipe finalizada para entre de sexta
pagina principal atualizada. imagens e formatação acrescentados
Faq e Quem somos, prontos para avaliação da equipe.
pagina equipe padronizada. Versão final para revisão
Copy link
Contributor Author

@jvpalves jvpalves left a comment

Choose a reason for hiding this comment

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

Oi, gente! Esse é o review do projeto de vocês. Os comentários mais pontuais estarão ao longo do código.

Em linhas gerais, eu gostei bastante do site de vocês! Está muito bonito e bem escrito, quase não tive o que comentar haha. O uso de HTML5 está óitmo, com as tags semânticas usadas corretamente, mas com algumas tags descontinuadas. No entanto, no geral está muito bem diagramado. Gostei muito da navegação e todo o esmero dado em todas as páginas O CSS também está excelente, com bom uso dos seletores, classes e Ids. Uma coisa que precisa ser destacada é a devoção de vocês ao projeto. Estava acompanhando o progresso e as conversas entre vocês, e acompanhando os commits. Achei incrível o quanto se uniram para resolver os problemas. Estão de parabéns! Até a próxima!

}

nav li{
/* border-right: 2px solid #698f69; */
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Se lembre(m) de remover código comentado (código morto)

right: 20px;
top: 20px;
color: #3b4f66;
background-color: #5b6c80;;
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Aqui temos um ;;. Isso pode encrencar na hora de renderizar o estilo.

}


.pessoas{
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Cuidado com a mistura de inglês com português na nomeação de seletores.

<form action="">
<div>
<label for="safe">Modalidade do seguro:</label>
<select name="safe" id="safe" required>
Copy link
Contributor Author

Choose a reason for hiding this comment

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

value volvo. Seria melhor definí-lo sem values, ou valores distintos

<section>
<div>
<span>Para ser seguro, não precisa ser difícil.</span>
<span><strong>Na 4Safe você consegue o melhor que o mercado pode
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Aqui seria melhor não usar mais tags como o strong.

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.

4 participants