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

vanilla CSS and html with flexbox

Mel 225

@Mel1207

Desktop design screenshot for the 3-column preview card component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


Feel free to comment on my very first design challenge at frontendmentor.io

is my media queries right ? does my css good enough to display expected design?

Community feedback

@syedalimansoor

Posted

Hello Mel!

You've done great, but I noticed a few issues if you don't mind me listing them.

  • The Learn More buttons do not have :hover states
  • The third card says SUVS instead of LUXURY
  • The cards' headings don't use the heading font given in the challenge
  • You start to see some cropping issues at a screen width of around 850px, you can fix them by settings your container's width to be a relative value

You can also check out my solution for reference. I hope this helps! Please upvote my comment if it does!

1

Mel 225

@Mel1207

Posted

@alimansoor-create

Thank you so much bro, I highly appreciate your feedback. I noted all the issues and yes this can be improved. This really helped me a lot for getting better at observing small details on my code. Thank you :)

1

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