Article preview component solution

Community feedback

• 

  Fraser Watt•1,790

  @fraserwatPosted almost 3 years ago

  This looks really great - very clean, especially at desktop width!

  Couple changes:

  • at mobile widths, you'll want to remove the width: 24rem on .card__image, as if you move the screen to 600-750 ish px wide the picture doesn't stretch across the whole div
  • Likewise, on your (max-width: 774px) media query, get rid of the height: on .card__body - you tend not to want to explicitly define the height in divs with content in them, and it's leading to a lot of white space around this width. This will introduce a padding issue, which you can also solve by removing the padding-bottom: 0; line from the same element.

  Keep it up!

  Marked as helpful

  1

  ---

  

  Juan Pablo Urra•230

  @ojitxslmlPosted almost 3 years ago

  @fraserwat Thanks! I will try those changes 🙌

  0