3-column preview card component - css grid

Community feedback

• 

  Daniel 🛸•44,310

  @danielmrz-devPosted 11 months ago

  Oi, Natalia!

  Seu projeto ficou ótimo!

  Tenho uma sugestão que pode ajudar:

  • Ao invés de usar margin pra reduzir o tamanho do card, aplica max-width a ele. Dessa forma, ele continuará responsivo, mas só crescerá até certo limite. Esse limite é o valor que você especificar no max-width.

  Utilizar valores percentuais (%) ou baseados no tamanho da viewport (vh e vw) pra margens, paddings, width e height pode causar bugs, além de fazer o card crescer demais em alguns casos e de menos em outros. Na dúvida, utilize rem como medida.

  Espero que ajude!

  Fora isso, seu projeto está excelente!

  Marked as helpful

  1
• 

  raya•2,870

  @rayaattaPosted 11 months ago

  Hi 👋 ÑataliaFracasso, congratulations on completing this challenge 🎉 I have some suggestions you might find useful.

  1. Every html document should have it's main content wrapped inside a main tag . You should replace <section class="caixa"> with <main class="caixa">

  2. It is much more advisable to include the link to your fonts inside your html instead of importing them in css.

  3. since the images are only decorative,their alt attribute should be left empty i.e alt = " " I hope this helps

  Other than that your solution is awesome👏 ✌️

  Marked as helpful

  1