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

Mobile first Responsive design

rosemutai 220

@rosemutai

Desktop design screenshot for the Product preview card component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Community feedback

Boris 2,870

@mkboris

Posted

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 helpful

0

@Supa-Thobile2

Posted

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