Reactions html,css,js

Community feedback

• 

  P

  David Tsanava•50

  @datotsPosted 4 months ago

  Hello, thanks

  0
• Account deleted

  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