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
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


feedback is appreciated.

Community feedback

Bryan Li 3,550

@Zy8712

Posted

Your site looks pretty good in mobile and desktop layout. Good job!

The only modifications I'd make to your site is changing the breaking point of your website (when if changes from desktop to mobile layout) because right now when I shrink my browser window it causes the contents of the page to be super squished together. I think it would be better for it to be in its mobile layout when its this size.

One other thing I'd change is not needing an alt description for all of the stars. This is due to the fact that you mainly want screen readers to read the important information on the page. So if you include an alt for all of them, it'll read "stars" outloud repeatedly over and over. The alt should mainly be used for images that proved some form of important information.

Hope you find this feedback useful 👍

Marked as helpful

1

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