3 Column Preview

Community feedback

• 

  Rio Cantre•9,690

  @RioCantrePosted almost 3 years ago

  Hello there! Nice work in completing this project. Looking at your solution, I would suggest the following for you...

  • Wrap the whole content design with specific tag like main for readability
  • Instead of wrapping each part individually with div, alternative way is like this...

  <div class="card-1 col-3" id="card">
        <img src="images/icon-sedans.svg" alt="Sedans">
        <h2>Sedans</h2>
        <p>Choose a sedan for its affordability and excellent fuel economy. Ideal for cruising in the city  or on your next road trip.</p>
        <button type="submit" class="btn" onclick="window.open('')" class="request-callback" style="color:#E28525;"><p>Learn More</p></button>
  </div>
  

  • Remove flex-direction: row; in .design rule set, add justify-content: center;, align-items: center; and margin: 6rem auto;
  • Remove margin-left: 200px; and margin-top: 200px; in .card-1, .card-2 and .card-3 rule sets
  • Increase the breakpoints with 842px for the responsive design
  • Remove margin: 50px 0 0 40px; in .card-1, .card-2 and .card-3 rule sets inside the media query max-width:400px

  Hope this helps and Keep it going!

  Marked as helpful

  1

  ---

  

  Mohammad Hemal•150

  @Cold-ThunderPosted almost 3 years ago

  @RioCantre Thank you Sir!💝 I hope that it will be helpful for me...☺️

  1