Kamasah-Dickson• 5,610
@Kamasah-Dickson
Posted
Your solution looks great good job Keep coding👍
Marked as helpful
0
Design comparison
Solution retrospective
i would like tips to improve responsive part, thanks
Community feedback
Mo'men Kamal• 130
@momenkamal221
Posted
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 helpful
0
Rony• 420
@romzc
Posted
@momenkamal221 thanks!!!!, ythanks, your solution it's cool, i will check your code : )
0
Liam R• 425
@Cooly-o-Cats
Posted
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 helpful
0
Please log in to post a comment
Log in with GitHub
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