@AdrianoEscarabote
Posted
Hello Favor Ejiofor, how are you?
I really liked the result of your project, but I have some tips that I think you will like:
1- Document should have one main landmark, you could have put all the content inside the main
tag click here
2- Page should contain a level-one heading, you could have changed h3
to h1
[click here](https://dequeuniversity.com/rules/axe/4.3/page-has-heading-one? application=axeAPI)
I noticed that the background image is dropping a lot at higher resolutions, to fix this we can do the following:
body { background-size: 100% 50vmin; }
the rest is great! Hope it helps... π
Marked as helpful
@Fave334
Posted
@AdrianoEscarabote I noticed the thing about the background image and I have fixed the problem; updated my repository but thanks for the feedback really helpfulπππππ.
@AdrianoEscarabote
Posted
@Fave334 no problem, keep coding!