vanilla CSS and html with flexbox

Community feedback

• 

  Syed Ali Mansoor•460

  @syedalimansoorPosted over 3 years ago

  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

  @Mel1207Posted over 3 years ago

  @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