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

solution to the simple card component challenge

@emmecee

Desktop design screenshot for the Product preview card component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Community feedback

@kevorklepedjian1

Posted

Congrats on the work. It looks good, i would advise you to style the text like the one in the design with the help of padding, other than that it looks awesome

Marked as helpful

0
pmigix 110

@pmigix

Posted

Congrats on tackling the challenge!

I might have some useful suggestions you can play around with!

1.Don't wrap content in unnecessary div tags, you've already done a great job of wrapping your content in separate sections, get rid of <div class="image-container"> and <div class="info-container">

2.Your image shrinks before hitting the breakpoint, so to make image fill up the section add additional properties to your .desktop-img { height: 100%; object-fit: cover; } and try setting the breakpoint at 768px

3.To stop your content from continuously growing on larger screens, on your .main-container instead of setting the width:50%;, try max-width:40rem;

I hope any of this helped in some way. Best of luck in further challenges!

Marked as helpful

0

@emmecee

Posted

Thank you! I still struggle to structure my html, this really helped alot.

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