3 column preview card component main using flexbox

Community feedback

• 

  Dimitar Radev•930

  @Mitko90Posted over 1 year ago

  Hi, nice solution

  It looks great. Almost pixel perfect.

  I have some suggestions:

  • Your solution generated accessibility reports.

  1. "Document should have one main landmark" - wrap the whole page in the <main> tag.
  2. "Page should contain a level-one heading" - add a visually hidden <h1>. You can check my submission here to see how I've done it

  • When the user clicks on the button they should be directed to a different part of the site. In that case the <a> tag should be used instead of the <button>

  • The three buttons are the same and also have the same :hover effect so maybe there is no need to use different classes for them. One class for all the buttons should be just fine.

  • I have a 1440px laptop and when I look at your site it gives me the mobile version. Maybe adjust the min-width of your @media a little and I suggest using em or rem units there instead of px

  Above all your solution looks great.

  Hope you found something helpful.

  Have a nice day

  Marked as helpful

  0