Skip to content
Submitted 4 months ago

Product preview card component solution

Prashant Uniyal•170
@mod-prashant
A solution to the Product preview card component challenge
Preview (opens in new tab)Code (opens in new tab)

Solution retrospective

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

I would appreciate any help in making the code more professional and any improvements you think I might need

Code

Please log in to post a comment

Log in

Community feedback

  • IO•710
    @i000o

    Hi!

    Your HTML looks good - You've made semantic choices. I would only say, you could replace <span> with a heading of your choice, like a <h2>. Always use <div>/<span> as a last resort - but in your <div> case, it makes sense to create the flex alignment.

    Your CSS also looks clean - As your projects get more complex and the CSS doc longer, you can consider using a pre-processor to help organise your approach to support clarified thinking when problem-solving! It has helped me a lot when things got confusing with more content.

    I think in the design, the product title has a hover state, too. But your solution looks great, otherwise. Well done.

    Marked as helpful
  • P
    Ruben Bardok•180
    @Rubenbard

    Hey Prashant, looks very good. I think were in the same boat, i also wonder if my code looks professional, I don;'t have a clue. So can't help you with that, but good job.

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