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

Reactions html,css,js

P

@datots

Desktop design screenshot for the Results summary component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Community feedback

@weldu0

Posted

hey David I was reviewing your code and noticed an opportunity for improvement.

  1. Use display: flex for the body like this:
    body {
      display: flex;
      flex-direction: column;
      justify-content: center;
      height: 100vh;
    }

This will make it easier to center the card-container. If you want to learn more about Flexbox, here is a great MDN doc. If you want a fun way to learn, Flexbox Froggy is great—it's what I personally used while learning Flexbox.

One thing I do not understand is why you chose to use JS for this simple UI?

Overall, it looks great; it just needs a bit of adjustment.

You're doing a great job! 👍 keep coding!

0

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