Skip to content
Submitted 10 months ago

Product preview card - Switching images techniques ?

Ynotlocin1•80
@Ynotlocin1
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Product preview card component
  • HTML
  • CSS
1newbie

Design comparison

SolutionDesign

Solution retrospective

Hello ! No particular issue with this challenge, perhaps my way of changing the header image from the mobile to the desktop part is not optimal. Could someone please notify me? And as always, if there are any suboptimal aspects in my code, I invite you to let me know to improve. I don't want to remain confined in my mistakes :) Thanks in advance.

Community feedback

  • Justin Connell•720
    @justinconnellPosted 10 months ago

    Your solution looks great!

    As per your request, I can suggest using the <picture> element - this element is ideal for handling 'art direction' and image src sets that target specific media queries with image sizes. This element also allows for scaling images for high resolution screens where you need higher pixel density to keep the user experience consistent with the technology they have.

    I hope this is useful

    Great work! keep it up.

    Marked as helpful
    0
  • Roraima Lugo González•510
    @roraima1986Posted 10 months ago

    Congratulations, my only recommendation for mobile design would be to put a padding-top and padding-bottom to separate the content from the edge

    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