Yazdunā¢ 1,310
@Yazdun
Posted
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 thismain
tag, must have alevel one heading
which tells what this page is about, thish1
can be hidden. your html containsmain
tag so just add ah1
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 tofont-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
Marked as helpful
1
Ajiboso Adeolaā¢ 70
@comfort-deola
Posted
@Yazdun Thank you, your suggestion was helpful
0