Desktop First Approach Using CSS Flexbox

Community feedback

• 

  P

  tediko•6,680

  @tedikoPosted over 3 years ago

  Hello, Arda! 👋

  Good job on finishing another challenge! What I can suggest you is:

  • Read about semantic. Semantic elements lead to more consistent code, they are easier to read and improve accessibility.
  • Since your .star image is decorative your alt text should be provided empty (alt="") so that they can be ignored by assistive technologies, such as screen readers.
  • However, continuing the stars topic. You repeat your HTML code with stars images which is unnecessary. Find the way to use only one star image for a .stars. You can do it easily by using background-repeat approach in your css.

  Good luck with that, have fun coding! 💪

  0

  ---

  

  Arda•85

  @ardaberrunPosted over 3 years ago

  @tediko Hello 👋. Thank you for your great feedback. I am appreciate.

  0