Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Html , Css

@Paulo-Borges

Desktop design screenshot for the Results summary component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


Terminei.. Muito legal mexer com as cores, com as transparências. Duvidas são muitas .. Quem puder me ajudar a melhorar, espero os comentários..

Community feedback

Daniel 🛸 44,270

@danielmrz-dev

Posted

Fala @Paulo-Borges!

Seu projeto ficou ótimo!

Tenho algumas sugestões para melhora:

  • Ao invés de usar margin pra tentar mover o card pro centro da página, você pode adicionar isso aqui ao body:
body {
    min-height: 100vh;
    display: flex;
    justify-content: center;
    align-items: center;
}
  • Adiciona também um hover effect no botão assim como cursor: pointer. Isso dará ao usuário uma indicação visual de que é um elemento clicável.

  • E por último, tenta utilizar mais as tags semânticas. Por exemplo, você poderia substituir sua div.container por main.container e por aí vai. Usar as tags semânticas melhoram a otimização SEO do seu projeto, assim como a acessibilidade.

Espero que ajude!

Fora esses detalhes, seu projeto ficou excelente!

Marked as helpful

0

@Paulo-Borges

Posted

Obrigado, @danielmrz-dev! Vou fazer essas mudanças..

1

Please log in to post a comment

Log in with GitHub
Discord logo

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord