Four card feature section

Community feedback

• 

  sonickonic•700

  @sonickonic

  Posted over 5 years ago

  Congrats on submitting your first solution! it looks amazing!

  You've currently got two h1 elements, it will be better to use it once as it can cause accessibility issues. Two ways to solve it:

  1. Wrap all of that text as a single h1 and use a span element inside it to separate the text.
  2. Use h1 for the main heading and h2 for the sub-title.

  Although it is a pixel perfect solution for the wide-screen, the mobile version is slightly off. Try next time to use min-width media queries instead of max-width. Starting from the mobile-first can lead to more simple solution)

  Good job!

  33