Four card feature section using HTML ,CSS

Community feedback

• 

  Yazdun•1,310

  @Yazdun

  Posted over 3 years ago

  Hello Ajiboso and well done 👏 ! Here are my suggestions:

  • On ACCESSIBILITY issue : each page should have at least one main tag which wraps the whole page's content and this main tag, must have a level one heading which tells what this page is about, this h1 can be hidden. your html contains main tag so just add a h1 to `main and your accessibility issues will be gone. Don't forget to generate new report though !
  • for font family, it seems like you've forgot to add comma font-family: 'Poppins'sans-serif;, make sure to change it to font-family: 'Poppins', sans-serif; so your fonts load correctly.
  • I think box shadow is too thick, I would've used something along the long with box-shadow: rgba(149, 157, 165, 0.2) 0px 8px 24px;
  • Use prettier extenstion on your IDE to format your code.

  ✅ Also I opened a pull request to your github which take care of above issues

  I hope this was helpful

  11

  Marked as helpful