Using flexbox and grid layout

Community feedback

• 

  darryncodes•6,370

  @darryncodesPosted almost 3 years ago

  Hi Antonio,

  Really good solution and very responsive.

  With a few tweaks i think it could be perfect:

  • i'd remove height: 70%; to .card-container as it's creating a really long card on desktop, you'll need space things out inside the card a little more to match the design
  • it might be cool to look into button styling to help match the desired hover effects to the design
  • you could clear your accssibility/html report if you mad this <section class="container"> a <main class="container"> and this <div class="attribution"> a <footer class="attribution">. semantic html is really important and fundamental

  Hope that helps

  Marked as helpful

  0