Single price using CSS grid

Community feedback

• 

  T

  ApplePieGiraffe•30,545

  @ApplePieGiraffePosted almost 4 years ago

  Hello there, Victoria! 👋

  Well done on this challenge! 👍

  I suggest adding a border-radius to the div.box and adding overflow: hidden to it so that none of the content inside it overflows the div.box and prevents the border-radius from being seen. Also, I think decreasing the width of the card component slightly in the mobile layout would make the card look a little bit better and the text inside it slightly nicer to read! 😉

  Keep coding (and happy coding, too)! 😁

  1
• 

  Carlos Contreras•145

  @carlosecontrerPosted almost 4 years ago

  Hi @victoriacesar, Congratulations, it looks great!

  In my case I did it using flexbox and it worked, but I had "problems" with the semantics of the html...

  I made my main box a flex container and then aligned and justified the content in the center.

  My code was something like this:

  container{
  display: flex;
  align-items: center;
  justify-content: center;
  }
  

  Maybe this will help you, bye!

  1
• 

  Rabin Gharti Magar•455

  @Rabin92Posted almost 4 years ago

  Hey @victoriacesar,

  Great job on this challenge!

  It's responsive and looks great on all screen sizes. To center the div you can add the height: 100vh; to .container in your media queries.

  Hope this helps & happy coding!

  1