Huddle landing page - Using CSS Grid and Flexbox

Community feedback

• 

  Lucas 👾•104,160

  @correlucas

  Posted almost 3 years ago

  👾Oi Leo Silva, tudo bem? Parabéns pelo desafio!

  Essa sua solução tá bem completa eu nem tenho nada pra adicionar, vou deixar só uma dica pra poupar seu tempo em novos projetos. O CSS padrão vem com um monte de comportamentos que deixam o código dificil de configurar, por exemplo, margens, imagens, animações e etc. Usando esse CSS RESET, você vai salvar um bom tempo usar esse arquivo que já reseta a maioria das coisas como margens, imagens, animações e etc. Vai salvar um bom tempo usar isso desde o começo: https://piccalil.li/blog/a-modern-css-reset/

  👋 Espero que essas dicas tenham sido úteis e continue codando!

  12

  Marked as helpful
• 

  Brian Schooler•440

  @superschooler

  Posted almost 3 years ago

  Hey Leo, awesome job on the project and great use of CSS Grid. Using Grid can help you save a lot of unnecessary <div>s in the code over flexbox, and I see you used it wonderfully.

  Because you did so well, I only have one small criticism. I'd probably add:

  .illustration {
  	align-self: center;
  }
  

  This will keep the picture centered vertically next to the text when your media query hits so it's not way up next to the logo on iPad-sized screens. I'd probably also change the media query to at least 700px since the image is so small between 577px and 700px/

  11

  Marked as helpful