Responsive four card feature section

Community feedback

• 

  Vanza Setia•27,775

  @vanzasetiaPosted over 2 years ago

  Hi, Luis! 👋

  Congratulations on finishing this challenge! 👏Your solution is responsive and looks great! It's great that you write the styling with the mobile-first approach and use em unit for the breakpoint for the media query! 👍

  Some feedback.

  • I highly suggest not changing the html font size because it can cause huge accessibility implications for those of the users with different font size or zoom requirements. So, I would recommend changing the body font size instead or even better setting the font size on the main element (that wraps all the page content in this case).
  • I would suggest using img element rather than inline SVG to make the HTML looks cleaner. 😉

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

  Marked as helpful

  1

  ---

  

  Luis paz•390

  @PazispeacePosted over 2 years ago

  @vanzasetia Thank you so much for the tips! highly appreciated

  0

  

  Vanza Setia•27,775

  @vanzasetiaPosted over 2 years ago

  @Pazispeace No problem! 👍

  1