3-column-preview-card HTML, CSS, FLEXBOX

Community feedback

• 

  Vanza Setia•27,775

  @vanzasetiaPosted over 2 years ago

  Hello there! 👋

  Good effort on making the site responsive! 👍

  There are some things that you can do to make the site better.

  • I would recommend making the body element as the flex container instead of the box element. Then, remove the margin from the box element. By making the body element as the flex container, the cards should be perfectly in the middle of the page.
  • I recommend setting a min-height on the body element to make the cards vertically center. Also, add some padding on the body element to prevent the card from touching the edges of the browser.
  • The car icons are decorative images. I can tell it because if they don't exist then there would be no missing information.

  That's it! Hope you find this useful! 😁

  Marked as helpful

  0

  ---

  

  coc0dev•190

  @coc0devPosted over 2 years ago

  @vanzasetia Thank you! I appreciate the help!

  0

  

  Vanza Setia•27,775

  @vanzasetiaPosted over 2 years ago

  @coc0dev You're welcome! 😉

  0
• 

  Hanningtone Monda•290

  @HanningtoneMPosted over 2 years ago

  This comment was deleted over 2 years ago

  0

  ---

  

  Vanza Setia•27,775

  @vanzasetiaPosted over 2 years ago

  @HanningtoneM Hey, Hanningtone Monda! Next time, please avoid asking the solution author to mark your comment as helpful. It can put unnecessary pressure on the solution author to mark your comment as helpful, which is best avoided.

  Also, It's best to help others without expecting anything in return. If the solution author marks your comment as helpful then it's a bonus. 🙂

  To learn more about what you should and you shouldn't do, you can read the Frontend Mentor community guidelines.

  0