3 card preview using flexbox

Community feedback

• 

  darryncodes•6,390

  @darryncodesPosted about 3 years ago

  Hi Garry,

  Well done, pretty much pixel perfect!

  A few considerations from me:

  • you could add transition: ease-in .3s; to your button styles for a smoother affect
  • you could consider using aria-hidden or adding alt-text to your images to help with the accessibility report. info on hidden
  • your <div class="wrapper"> should be <main class="wrapper">
  • i think your button mark up has gone a little awry <a href="#" id="btn-three"button class="about">Learn More</button></a>
  • it's best practice to have one <h1> on your design (a little tricky with this design as its more of a component that would form a bigger design)

  Happy coding!

  Marked as helpful

  1
• 

  Ravi Pratap Singh•110

  @cyberspatialPosted about 3 years ago

  Beautifully implemented solution.

  Your code has few issues!

  1. For the <a> tag you have mistakenly combined button and <a> tag together.
  2. U should use alternative text for the image in case the browser doesn't load the image. < img src="" alt="image-description-in-brief" >

  Good Luck!

  Marked as helpful

  1