Skip to content
Submitted almost 2 years ago

Product preview card component using CSS flexbox

Shraddha Vkarma•40
@Shraddha-V22
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

1- Assuming the actual width and properties of elements was a bit challenging. 2- I am unsure whether I have written an efficient code or not. 3- There are couple of doubts about best practices and whether I am practicing them or not.

Community feedback

  • Mariana Rainha•50
    @marianarainhaPosted almost 2 years ago

    Hello Shraddha !

    You should set hover and focus states for interactive elements, like the button, to change it's background color and the cursor, for exemple.

    Marked as helpful
    0
    Shraddha Vkarma•40
    @Shraddha-V22Posted almost 2 years ago

    @marianarainha thank you. I will do that.

    0
  • suhaib52•280
    @suhaib52Posted almost 2 years ago

    Your code is looking clean. It would be great if you use classes as compared to the ids in the html, because use of classes considered best practice as compared to the ids.

    Marked as helpful
    0
    Shraddha Vkarma•40
    @Shraddha-V22Posted almost 2 years ago

    @suhaib52 Yes, I will remember that. thank you!

    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