Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Product-review V1

Ikapocc 170

@Ikapocc

Desktop design screenshot for the Product preview card component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

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

@izzatrazab

Posted

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

@Ikapocc

Posted

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

@izzatrazab

Posted

@Ikapocc You're welcome, hope that helps.

0

Please log in to post a comment

Log in with GitHub
Discord logo

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