@AdrianoEscarabote
Posted
Hi CyPhEr420, how are you? I really liked the result of your project, but I have some tips that I think you will enjoy:
As you know this project is based only on one page component, so no h1 is needed! because we don't know if this page will have a more important component, but it's always good to prevent accessibility errors so I think it would be good for you to add an h1
in this component, besides being a good practice for when you are developing larger sites, don't worry forget about h1
.
<h1 class="qr-title">Improve your front-end skills by building projects</h1>
Add a padding to the body, so that the content in lower resolutions doesn't hit the edge of the screen!
The rest is great!
I hope it helps... π
Marked as helpful
@CyPhEr420
Posted
@AdrianoEscarabote thnx for that am just starting to focus more on the accessibility