Article Preview Component using flexbox (and a small amount of grid)

Community feedback

• 

  Mubaraq Wahab•480

  @mubaraqwahabPosted over 4 years ago

  Hey this is good, well done!

  I have two comments:

  1. The avatar seems stretched on desktop. I think giving the .user-avatar__container a width equal to its height should fix it (width: 4rem; perhaps).

  2. Second thing, I think it would be good to have the share button as a <button> not a link. As a link, clicking it scrolls the page back to the top.

  1

  ---

  

  Anthony•205

  @AFLazaroPosted over 4 years ago

  @mubaraqwahab

  1. That's interesting, it looked fine on Firefox but I checked in Chrome and it was stretched out as you said. This has been fixed.

  2. Excellent suggestion, I've changed the anchor element to a button.

  Thanks for the feedback!

  1

  

  Mubaraq Wahab•480

  @mubaraqwahabPosted over 4 years ago

  @AFLazaro You're welcome :)

  0