3 Column Preview Card Component

Community feedback

• 

  Jeuri Morel•1,385

  @JeuriMorelPosted about 1 year ago

  One general tip is to add cursor: pointer; to your buttons and/or links. It's a small change, but leads to better user experience.

  Also, don't just check what the site looks like on mobile and at full screen widths. Look at how it functions on medium sizes as well. Your solution's layout starts to break down in the 500px-900px range. The cards get all squished up and sometimes you even lose content, like the button, because it gets so narrow.

  Marked as helpful

  2

  ---

  

  Ahmad Hassan•310

  @Ahmadhassan0Posted about 1 year ago

  @JeuriMorel Thanks

  Next time I'll keep in mind to add cursor: pointer; and also I'll make it responsive for medium devices😊

  0
• 

  Rebecca Padgett•1,840

  @bccpadgePosted about 1 year ago

  Hello @Ahmadhassan0. Congratulations on completing this challenge!!!🎉

  When you complete Frontend Mentor projects, be sure to check the Accessibility report and HTML validation report for errors. Looking at those reports gives you great feedback on you are doing wrong and how to improve your solution. I don't want you to keep making the same mistakes over and over with each project you submit.

  Here is an article about buttons vs. links and when to use them depending on the purpose.

  Adding an aria-label on each link will explain the purpose as well.

  • aria-label="learn more about sedan cars"

  Hope you find this useful.

  Marked as helpful

  1

  ---

  

  Ahmad Hassan•310

  @Ahmadhassan0Posted about 1 year ago

  @bccpadge Thanks again

  I'm not familiar with accessibility, but today I'll learn about it and will make every project with good accessibility🙂✔

  And Sorry for stealing your screenshot section in readme file😅

  0