All comments

• 

  Novidicus Titan•60

  @NovidicusTitanSubmitted about 3 years ago

  I have made a video about how i did this project on my youtube chanel if you want to check it out, thanks

  

  first-project-responsive-order-summary-using-grid, only html & scss

  2

  10
  

  Saurav Singh Khekhaliya•190

  @sskhekhaliyaPosted about 3 years ago

  hey, I just looked at your site, there are some suggestions:

  1. There is no need to using border-radius: 20px 20px 0px 0px; for image, instead of it just apply overflow: hidden; to .container. So it will hide the corners of images also.
  2. It will be good to give a little padding-bottom to the card.
  3. Card's margin-top:40px; didn't make it vertically centered and if someone will see your site on the big screen then the card is not able to be vertically centered. So for it, you can use display: flex; for body and then set its direction to column (flex-direction: column;) and give a height to the body 100vh.

  Marked as helpful

  0

  This comment has received 1 replyShow