4Card Challenge using Flexbox, SASS

Community feedback

• 

  P

  tediko•6,680

  @tedikoPosted over 3 years ago

  Hello, Rizwan Mustafa! 👋

  Congrats on finishing another challenge! Your solution responds well and looks good. Here are few suggestions from me:

  • You can use just one heading element for .light. Just wrap text that you want to be bold with <span> and style it.
  • Since your icons image is decorative your alt text should be provided empty (alt="") so that they can be ignored by assistive technologies, such as screen readers.
  • Read about Sass 7-1 pattern to keep your file management orginazed.

  Good luck with that, have fun coding! 💪

  1

  ---

  

  Rizwan Mustafa•155

  @rizwanmustafaPosted over 3 years ago

  @tediko Thanks alot. Will look into the pattern!

  0