3-column preview card using Flexbox

Community feedback

• 

  Raymart Pamplona•16,100

  @pikapikamartPosted about 3 years ago

  Hey, awesome work on this one. Desktop layout looks great, the site is responsive and the mobile state looks great as well.

  Some suggestions would be:

  • h1 should only be limited to 1 usage per page of the site. Change those h1 into h2.
  • Since you are removing the h1, you will need to create another but it will be only a screen-reader only h1. Meaning it will be hidden visually but present for screen-reader users. On this, the h1 would have like sr-only class and the text-content should describe what is the main-content is all about. The h1 could be placed as the first text inside the main. Have a look at Grace's solution on this one. Inspect the layout and see the usage of h1 as well the stylings applied.
  • learn more is more suitable to be a link, since on a real site, this would act as a page link where the users can "learn more" about a specific car.

  Aside from those, great job again on this one and congrats for your 3rd challenge in here^^

  Marked as helpful

  0

  ---

  

  Han•340

  @hcxwebPosted about 3 years ago

  @pikamart Hello Raymart, Thank you very much for the valuable and detailed suggestions! I just learned something new and will try to make it better.

  1