3-column-preview-card-component

Community feedback

• 

  Aman Singh Bhogal•1,010

  @asbhogalPosted over 1 year ago

  Hi Rolando, this is a good attempt and I can see you've tried to remain faithful to the mockups. I've listed below ways in which to improve this, incl. responsively:

  • Your columns should be individual grid items inside a parent grid to maintain what we call the 'intrinsic' (ie. fixed) width values which flex does not (flexbox is design, as the name suggests, for content that allows greater 'elasticity', if you will). Here's a good video by Kevin Powell on the difference between the two, and when to use which Link
  • Remove the margin declaration you have on the main component as this is causing issues also in your positioning. To centralise this, you can add a grid also to the parent or use flex to vertically and horizontally position
  • Locally host your Google Fonts for privacy and performance. Here's another useful link explaining why this is the case, and how to achieve it Link

  Hope this helps!

  0