Skip to content
Submitted 10 months ago

Responsive Product preview card component

Lucie•120
@luciehelias
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

What are you most proud of, and what would you do differently next time?

Everything because I start to be more efficient and more faster.

What challenges did you encounter, and how did you overcome them?

On this one I didn't encounter any issue.

What specific areas of your project would you like help with?

Everything that can be help to improve my code and my way to do it.

Please log in to post a comment

Log in with GitHub

Community feedback

  • P
    Daniel 🛸•44,780
    @danielmrz-dev

    Hello there!

    Congrats on completing the challenge! ✅

    Your project looks great!

    I have a suggestion about your code that might interest you:

    📌 You can use the <picture> tag when you have different versions of the same image.

    Using the <picture> tag will help load the correct image to the user's device, saving bandwidth and improving performance.

    Example:

    <picture>
    <source media="(min-width: 768px)" srcset="{desktop image path here}">
    <img src="{mobile image path here}" alt="{alternative text here}">
</picture>

    I hope this helps!

    Other than that, excellent work!

  • P
    Paul Olynek•330
    @ePaulo

    Your HTML and CSS look good. So does your UI based on the design files. I did notice the spacing between the lines is different from the provided design. I've been using the CSS styling "line-height" [https://developer.mozilla.org/en-US/docs/Web/CSS/line-height] to control spacing between lines. Give it a try... I find setting line-height very useful. :-)

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