Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Desktop First Approach Using CSS Flexbox

Arda 85

@ardaberrun

Desktop design screenshot for the Social proof section coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


Any feedback will be appreciated. Thank you :)

Community feedback

P
tediko 6,580

@tediko

Posted

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

@ardaberrun

Posted

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

0

Please log in to post a comment

Log in with GitHub
Discord logo

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord