-
Notifications
You must be signed in to change notification settings - Fork 0
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
base: correcao-projeto
Are you sure you want to change the base?
PR de correção #13
Conversation
pagina equipe CSS html e arquivos
cores corrigidas
equipe finalizada para entre de sexta
index com texto
entrega incompleta
pagina principal atualizada. imagens e formatação acrescentados
Semana2 projeto
Faq e Quem somos, prontos para avaliação da equipe.
pagina equipe padronizada. Versão final para revisão
There was a problem hiding this 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; */ |
There was a problem hiding this comment.
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;; |
There was a problem hiding this comment.
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{ |
There was a problem hiding this comment.
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> |
There was a problem hiding this comment.
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 |
There was a problem hiding this comment.
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
.
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
O que não funciona
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