3 Column preview card solution

Community feedback

• 

  Vanza Setia•27,775

  @vanzasetiaPosted over 2 years ago

  Excellent work on this solution, Makson! Your solution looks great! 👏

  The site looks good on both portrait and landscape mobile views! It also looks great on my desktop view! 👍

  There are some areas that can be improved.

  • I would suggest leaving the alt empty because all the icons are decorative images which means that if the images don't exist, there's no missing content. 😉
  • Link elements might be a better choice for all the Learn More buttons. I would expect if I click one of those buttons I would be navigated to another page or a certain part of the page. So, what do you think it's going to happen if the user clicks the button?
  • By default, the body element has already full width so, I recommend removing the width: 100% from the body styling.
  • I highly recommend using rem or sometimes em unit instead of px. Using px will not allow the users to control the size of the page based on their needs.

  About the responsiveness, I would recommend writing the styling using the mobile-first approach. It often leads to shorter and better performance code. Also, mobile users won't be required to process all of the desktop styles. 👍

  That's it! I hope this information is useful! 😁

  Marked as helpful

  0

  ---

  

  Makson Douglas•260

  @Makson19Posted over 2 years ago

  @vanzasetia Thank you so much, I will make sure to adjust the code.

  0

  

  Vanza Setia•27,775

  @vanzasetiaPosted over 2 years ago

  @Makson19 You're welcome! 😄

  0