Skip to content
Submitted over 1 year ago

Product-review V1

Ikapocc•170
@Ikapocc
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

I struggle a bit with the IMG, there are some parts that I'm not too convinced about, like the use of the border radius. I know what the problem is, it's the use of the min (width), but I wanted to try this function. so if there is another way to solve this, any feedback would be nice.

Community feedback

  • Izzat Razab•70
    @izzatrazabPosted over 1 year ago

    Hi, firstly I am not really qualified to comment on people's work as I am a beginner myself. But I will try to give suggestions that may benefit you. I have read your code carefully but if I am still wrong in any way, do reply here. My comments require images, therefore I use google doc instead:

    click here to view the google doc

    1
    Ikapocc•170
    @IkapoccPosted over 1 year ago

    Hi Izzat, thank you very much for this intuitive explanation, yes I didn't like the way I applied the border radius even when I was doing a similar project with almost the same features as this one didn't have this problem and I was wondering why. For part of the grid it was a mistake on my part, I don't know why I didn't remove it, but thanks for pointing it out.

    0
    Izzat Razab•70
    @izzatrazabPosted over 1 year ago

    @Ikapocc You're welcome, hope that helps.

    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