Skip to content
Submitted 5 months ago

Product responsive

Ian Cardoso•90
@Ian-Cardoso
A solution to the Product preview card component challenge
Preview (opens in new tab)Code (opens in new tab)
Code

Please log in to post a comment

Log in

Community feedback

  • Rafael•120
    @rafael-study-repo

    Congratulations for the project! The layout is consistent and responsive—I'll just point out a few things to help improve it. Your HTML has a small indentation error in the .info div, but other than that, it's well structured. An alternative to your solution would be to use the <picture> and <source> elements for serving desktop and mobile layout images.

    As for your CSS, you made good use of background-image. I would personally prefer using responsive images (as mentioned above) for better control, but you did a great job. Another thing I’d consider is using relative units (like rem or em, for example) instead of px. In any case, good job.

  • vaibhu252frontend•120
    @vaibhu252frontend

    solution doesn't include semantic HTML.

  • vaibhu252frontend•120
    @vaibhu252frontend

    nice solution

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