Skip to content
Submitted 10 months ago

Mobile first Responsive design

rosemutai•220
@rosemutai
A solution to the Product preview card component challenge
Preview (opens in new tab)Code (opens in new tab)
Code
Couldn’t fetch repository

Please log in to post a comment

Log in

Community feedback

  • P
    Boris•4,110
    @makogeboris

    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
  • Supa-Thobile2•100
    @Supa-Thobile2

    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.

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