3 Column Cards with Flexbox & MediaQueries

Community feedback

• 

  Lucas 👾•104,340

  @correlucasPosted about 2 years ago

  👾Hello Fareem Mohamed, Congratulations on completing this challenge!

  Your solution its almost done and I’ve some tips to help you to improve it:

  1.You made your html structure entirely with div blocks but these div doesn't any semantic meaning, for this reason is better you use a better html markup improving your code, for example for each vehicle card you use <article> instead of the <div>.

  2.Replace the <div> you’ve used to wrap the card container with <main>, that in this case is the best tag and shows that this is the main block of content of this page.

  3.Think about using relative units as rem or em instead of px to improve your performance by resizing fonts between different screens and devices.Anyhow, if we want a more accessible website, then we should use rem instead of px. REM does not just apply to font size, but to all sizes as well.

  ✌️ I hope this helps you and happy coding!

  Marked as helpful

  0