Stats Preview Card

Community feedback

• 

  jerry•300

  @realsalePosted over 2 years ago

  Yo! @MrSkyner,

  Congrats submitting your solution for this design.

  The obvious thing that might need refinement is your alignment, I've noticed you haven't use flex nor grid and hardly relied on margin and some fixed dimensions.

  I recommend you to try and learn flex and/or grid as these are the modern layout approach Beginners guide to flexbox | Beginners guide to grid

  Also you don't have to explicitly define the containers height, Just set the similar width as the design, and let the image take the half of it and leave image height: auto, This will keep the image proper aspect-ratio

  Marked as helpful

  1
• 

  Purnima Kumar•390

  @purnimakumarrPosted over 2 years ago

  Hi Skyner, your solutions look really good. But it isn't centered properly. Here's what you can do..

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

  And get rid of the margin on the card component.

  1
• 

  alfred thompson ovie•820

  @alfredthompsonOviePosted over 2 years ago

  Hi Mr skyner, looks nice, but you could do with a little bit of padding to the contents

  1