four-card-feature-section-master

Community feedback

• 

  P

  KonradJam•1,120

  @KonradJamPosted 16 days ago

  Hello @Eng4atou!

  Good job! I have some fixes for your code:

  • Try using HTML5 semantics more efficiently.

  <main> => <div class="container"
  <section class="content-container"> =>  <header class="content-container">
  <section class="card-container"> => <main class="card-container">
  <div class="card [name]-card"> => <section class="card [name]-card">
  

  You can read more about HTML5 semantics in my post on LinkedIn

  • Use relative units like rem or em instead of px. My publication about Relative Units on LinkedIn
  • Add some padding to the main container, because on the 1024px resolution, card are touching the edges of the page.

  Marked as helpful

  0