Price Component using CSS Grid

Community feedback

• 

  Adriano•38,250

  @AdrianoEscarabotePosted about 4 hours ago

  Hello Rafiza Hanif Rahd, how are you? I was really pleased with your project, but I’d like to offer some advice that might help:

  Using Flexbox or Grid on the body to center elements ensures a more responsive and adaptive layout, fitting different screen sizes seamlessly. It avoids manual calculations and constant adjustments needed with margin, padding, or absolute positioning. These techniques provide more consistent alignment and simplify the code.

  flexbox:

  body {
      display: flex;
      justify-content: center;
      align-items:  center;
      min-height: 100vh;
  }
  

  grid:

  body {
      display: grid;
      place-content: center;
      min-height: 100vh;
  }
  

  The rest is spot on.

  Hope it’s helpful to you. 👍

  0
• 

  hitmorecode•6,660

  @hitmorecodePosted 1 day ago

  Nice well done looks good. I don't understand what kind of help you need fixing the background color. It looks OK to me. Just a few thing that I want to point out.

  • On the body it's good practice to use min-height instead of just height.
  • Why are you using so many media queries? You can do all this with just one media query.

  I hope you find this helpful. Keep it up 👌👍

  0