Ebenezer Silva
@Bene001700All comments
- @AdnaneBcSubmitted 28 days ago@Bene001700Posted 4 days ago
Para ajudar vai um feedback:
-
Tente usar mais tags semânticas: Por exemplo. No contêiner pode utilizar <main> tag.Também envolver todo conteúdo principal na <article> tag. Para imagens pode utilizar <figure> tag semântica para imagem.
-
Em questão de layout: O documento já vem por padrão uma margin e padding defenida. Remova definindo para 0 e assim você tem mais controle nos espaçamento e tamanho dos elementos. Para remover basta utilizar o seletor universal * no css.
-
Aplique o conceito de
mobile first
para construção. Use tambémmedia query
para mudança de layout. As media query ajuda a controlar a parte responsiva.
Marked as helpful0 -
- @MarinaPerezRSubmitted 18 days agoWhat are you most proud of, and what would you do differently next time?
I'm feeling more comfortable using flexbox.
What challenges did you encounter, and how did you overcome them?I get a bit confused when using @media for responsiveness.
What specific areas of your project would you like help with?Any feedback is always welcome!
@Bene001700Posted 17 days agoBom trabalho! Só uma sugestão. Tiraria o "height : 100vh" do body. Não sei por qual motivo ele faz com que o conteiner fique colado no topo. Mas é um bom trabalho. Parabéns.
Marked as helpful1 - @4skinSkywalkerSubmitted 23 days ago
- @rohit7318Submitted 23 days ago@Bene001700Posted 23 days ago
Muito bom!
Tente seguir mais fiél possível o layout do figma. Gostei como trabalhou a responsividade do layout com max-wdith e porcentagem.
1