Skip to content
Submitted 3 months ago

Product Preview with HTML and CSS

#pure-css
kunal90803•490
@kunal90803
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?

Proud of completing my project and also learn some new stuffs.

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

Problem arises during change of image with change in screensize so i use picture tag with srcset and media property.

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

Please provide feedback for this project that could help me in further future projects.

Community feedback

  • P
    orrest•110
    @orrestPosted 3 months ago

    The preview seems good, but there do have some problems of the code as far as I concern.

    1. You comment some meaningless code, which could be safely deleted.
    2. You use the space to make the PERFUME has its letter spacing, but actually, there do have the letter-spacing CSS property.
    3. You use the inline style, which I don't think is a good option.

    BTW, the shadow thing looks awesome!

    If there's anything that I point out is not properly, please let me know.

    Marked as helpful
    1
    kunal90803•490
    @kunal90803Posted 3 months ago

    @orrest Thanx for reviewing my project .. and it really helps me in learning about new term i.e. letter-spacing. Surely gonna consider this review in future projects.

    1

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