3 column preview card component with html and responsive css

Community feedback

• 

  Luis Felipe•230

  @FelipinDRPosted almost 2 years ago

  You should not add an alt tag to the icons because they're just icons, also you should use "aria-hidden:true" to hide them from accessibility Good job finishing this, good luck with the next challenges

  Marked as helpful

  1

  ---

  

  BLESSING DAWODU•10

  @DammyDPosted almost 2 years ago

  @FelipinDR I'll make sure to put this feedback to use in my next challenge. Thank you.

  1
• 

  suhaib52•280

  @suhaib52Posted almost 2 years ago

  You did it in a very good manner. Your code is well structured and readable.

  It would be nice if you start using semantic html tags, these tags would reduce the accessibility issues.

  You can also use :root to define the colors and then call them by using var() in your stylesheet. When you use the colors as a variable than your css code will look more precise.

  Marked as helpful

  1

  ---

  

  BLESSING DAWODU•10

  @DammyDPosted almost 2 years ago

  @suhaib52 I really appreciate your feedback. I promise to put this feedback to use in my next challenge.

  Thank you.

  0