3-Col-Preview-Card-Component using CSS Flexbox

Community feedback

• 

  Melvin Aguilar 🧑🏻‍💻•61,140

  @MelvinAguilarPosted 11 months ago

  Hello there 👋. Good job on completing the challenge !

  I have other suggestions about your code that might interest you.

  • Consider adopting a Mobile First approach in your next project. This means designing for the smallest screens first and then scaling up as needed. This helps to create a more user-friendly and accessible experience for all users.

  • You should use only one <h1> tag per page. The <h1> tag is the most important heading tag, This can confuse screen reader users and search engines. This challenge requires that Sedans, SUVs and Luxury are headings, but you can use the <h2> tag instead of the <h1> tag. You can read more about this here 📘.

  • Not all images should have alt text. Car icons are for decoration purposes only, so they can be hidden from screen-readers by leaving its alt attribute empty. You can read more about this here 📘.

  I hope you find it useful! 😄 Above all, the solution you submitted is great!

  Happy coding!

  Marked as helpful

  3

  ---

  

  Dashaun Sutton-Harris•80

  @dashaunnPosted 11 months ago

  Thank you for your helpful response @MelvinAguilar!! 🙏

  Oh wow that's embarrassing 😅 I know better than to add more than one h1 element. Thank you for noticing that!

  Thank you for the advice and resources Melvin, this is all really good to know. I will try a mobile first approach in my next projects. Thank you again!! 😁

  1
• 

  Daniel 🛸•44,250

  @danielmrz-devPosted 11 months ago

  Hello @dashaunn!

  Your project looks great!

  On your questions:

  • Is the semantic HTML of my document okay?

  Yes. Only thing I'd do different is to use only the main tag, instead of both main and article. Unless you did that for a specific reason, there's no need to use both when just one can do the job.

  • Is there a neat way to measure out sizes of things? I find myself squinting and tweaking sizes a lot to match the design.

  Without the figma file, we all guess the sizes 😅

  • I would appreciate any tips on best practices for my CSS! (On media queries, organization, etc. )

  You CSS is great. I'd just increase your media queries breakpoint to show your desktop version only if the screen has at least 650px. Between 650px and 375px (your current breakpoint) your main content gets squeezed.

  I hope it helps!

  Other than those little details, you did an excelent job!

  Marked as helpful

  1

  ---

  

  Dashaun Sutton-Harris•80

  @dashaunnPosted 11 months ago

  Thank you so much for your thorough response @danielmrz-dev!! 🙏😄

  This is really helpful, I will make some adjustments to my project now. 😁

  1