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

flexbox ,grid

ubey_Sa 120

@ubeysaab

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


Firstly, I would like to express my gratitude for this wonderful platform. Secondly, I would appreciate your feedback on my solutions (application and code sections) so that I can improve my coding skills by writing cleaner code and becoming a better programmer. Thank you.

Community feedback

@Mriganka5137

Posted

  • I would suggest not use the image as background image property.
  • use the div with class "up" as the container to put the <img /> inside.
  • And refrain from using height: property. it will break the responsiveness.
  • Don't use padding in the "p" tag .
2

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