Skip to content
Submitted about 2 months ago

Responsive product preview card with flexbox

Gianmarco•200
@gianmromero
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?

Understanding of flexbox, I will use grid layout next time.

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

Improving my understanding about web elements.

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

About the asterisk property. I knew it's not a good practice but it works for me this time. What I can do to avoid this practice?

Community feedback

  • ramiadi•50
    @ramiadiPosted about 2 months ago

    Nice design and coding here! i see you are using alot of px. Thats not a problem but i suggest to use rem or em when it comes to margins, padding and font-sizes. Use rem/em when you want it to be adaptive to the scale of the system. For example, if you use rem or em the size will change which adds accessability for people that have it harder to read.

    thats all i got, keep it up!

    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