Design comparison
SolutionDesign
Community feedback
- @mkborisPosted 4 months ago
Hi rosemutai nice work , some suggestions
- Responsive images are handled using the
picture element
. Using CSS to hide and show images to display differently on various screen sizes is not part of the best practices. - Consider using a modern CSS reset at the start of the styles in every project. Like this one Modern CSS Reset. This will help reset a list of default browser styles.
- Media queries should be defined in rem or em not px.
Marked as helpful0 - Responsive images are handled using the
- @Supa-Thobile2Posted 4 months ago
Absolutely brilliant! The design has been translated to code beautifully.
The code base is easy to read and follow. The design has been translated well using semantic HTML.
The Issue I have is on the mobile view scroll. If the developer could work on remove any additional padding or margin for small screens.
0
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