@AdrianoEscarabote
Posted
Oi Cauê Souza, tudo bem?
Gostei muito do resultado do seu projeto, mas tenho algumas dicas que acho que você vai gostar:
Como você sabe, este projeto é baseado apenas em um componente de página, portanto, não é necessário h1! pois não sabemos se esta página terá um componente mais importante, mas é sempre bom prevenir erros de acessibilidade então acho que seria bom você adicionar um h1
nesse componente, além de ser uma boa prática para quando você está desenvolvendo sites maiores, não se preocupe, esqueça o h1
.
main {
min-height: 100vh;
}
Use min-height
em vez de height
, pois quando o conteúdo for além de main
o conteúdo acabará sendo ocultado! faça o teste aumentando a altura do article!
o resto é ótimo!
espero ter ajudado... 👍
@cauesooouza
Posted
@AdrianoEscarabote fica tranquilo que questão de acessibilidade e seo, esta bem otimizado, e enquanto ai height, não faz diferença pois se fosse uma pagina com outros elementos, nao seria feito dessa forma utilizando 100vh, ele seria em tamanho relativo, ocupando somente o espaço necessario, o 100vh foi utilizado somente para questões de centralização, obrigado pela sua ajuda mas não é necessario alterações
@AdrianoEscarabote
Posted
@cauesooouza boaa