Results Summary Component

Community feedback

• 

  Hassan Moataz•1,860

  @hassanmoaaPosted 9 months ago

  Hello @ViniCellist!

  Great Job solving the challenge mate *congrats * 🎉

  Some suggestions for improvements.

  Heading is defined with <h1> to <h6> tags. It is important to use headings to show the HTML document structure.

  <h1> headings should be used for main headings, followed by <h2> headings, then <h3>, and so on up to <h6>

  ➖➖➖➖

  • Consider increasing the width of the container also!

  Other than that you're good, keep up the good work!

  Marked as helpful

  1
• 

  Daniel 🛸•44,250

  @danielmrz-devPosted 9 months ago

  Fala @ViniCellist!

  Seu projeto ficou muito bom!

  Uma dica:

  • Evite usar valores baseados no tamanho da viewport (vh e vw) pra configurar a largura ou altura de elementos. Parece que ajuda na responsividade, mas na verdade não ajuda.

  Uma alternativa pra manter a adaptabilidade do elemento é configurar uma largura máxima pro elemento com max-width e um valor em pixels ou rem. Dessa forma, o elemento continua responsivo, mas só cresce até determinado ponto.

  Pra altura, nem precisa configurar valor nenhum, só basta colocar os elementos filhos dentro que o elemento pai se adapta pra fazer caber tudo dentro dele.

  Espero que ajude! 😊

  Marked as helpful

  1