Design comparison
Solution retrospective
i would like tips to improve responsive part, thanks
Community feedback
- @Kamasah-DicksonPosted over 2 years ago
Your solution looks great good job Keep coding👍
Marked as helpful0 - @momenkamal221Posted over 2 years ago
Great job Rony, your solution is responsive. but exactly after 800px screen width, I noticed that the stars overlap with the next text, set 'min-width' property to 5.3rem for the selector 'ul li span' and it will work fine, and I like how your starts are a background so you change the number of the stars by just changing the width of the 'span' tag 🤩 here is my solution to this challenge, I would be happy if it helps you https://www.frontendmentor.io/solutions/social-proof-section-solution-momen-kamal-3b4ZXldJA
Marked as helpful0@romzcPosted over 2 years ago@momenkamal221 thanks!!!!, ythanks, your solution it's cool, i will check your code : )
0 - @Cooly-o-CatsPosted over 2 years ago
Personally, I think that the site looks great and is very responsive. Be sure to keep all HTML elements inside either
<header>
,<main>
or<footer>
tags. That will fix your accessibility issues.Marked as helpful0
Please log in to post a comment
Log in with GitHubJoin 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