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 preview

Eliabe 200

@eliabedasilva

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


This was my first project on the platform and it wasn't much of a challenge I just had to study a little about flexbox and the implementation with github pages. Overall it was a very fun and very productive process.

Community feedback

Elaine 11,400

@elaineleung

Posted

Hi Eliabe, welcome to Frontend Mentor, and great job working on this challenge, as it looks really good! 😊

One suggestion I have is to use height: 100% for #image instead of the fixed pixel height you have; the reason is, in mobile view when I resize the browser and make the width smaller than 300px (I wasn't intentionally trying to do this, more of an accident but led me to discover this), the image separates from the text, and using height: 100% can keep that from happening here. You can also add a display: block to your img to remove that tiny almost 1px spacing that's underneath the image, which occurs due to img being an inline element and thus have properties like line height. Well done on the whole!

Marked as helpful

1

Eliabe 200

@eliabedasilva

Posted

@elaineleung I also encountered this problem of the image separating from the text, but I had no idea how to solve it. Thanks for the guidance.

0
Kenbak 40

@Kenbak

Posted

Congrats on your first challenge ser. It's very impressive. The card is the exact same size.

Instead on using margin top and bottom on the texts elements, I would put them all in a div, using flex and flex-column. Then add a justify-content-around and it should do the trick and be more symmetrical!

Also you can reduce a bit the line height on the H1 to make it perfect ;)

Marked as helpful

1

Eliabe 200

@eliabedasilva

Posted

@Kenbak thanks for the guidance

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