3 Colum Preview Card Solution

Community feedback

• 

  Vanza Setia•27,775

  @vanzasetiaPosted over 2 years ago

  Hi there! 👋

  Nice work on this challenge! It's great that the site is using the Normalize and all the page content lives inside a landmark! 👍

  However, there are some things that can be improved.

  • The icons are decorative images which means that if those icons don't exist then there will be no missing information. So, I suggest leaving the alt empty so that the screen reader users won't have to listen to the unrelated content.
  • I highly recommend using rem or sometimes em unit instead of px to make sure that everything can scale properly according to user needs. Using px will not allow the users to control the size of the page based on their needs.

  I hope you find this information beneficial. Happy coding! 😄

  Marked as helpful

  0

  ---

  

  GSD3v08•190

  @GSD3v08Posted over 2 years ago

  @vanzasetia Thank you very much for your feedback, I'll use more often rem and em units in next projects, getting used to them it's a little difficult at first and some times I forget to use them.

  0

  

  Vanza Setia•27,775

  @vanzasetiaPosted over 2 years ago

  @GSD3v08 You're welcome! 😄

  Take it step by step, maybe at first you want to only use rem for all font-size and after that, start using it on margin and padding, and until you've managed to use rem as your main unit instead of px.

  Hope my tips help!

  0