@AdrianoEscarabote
Posted
Hi Reza Jaber, how are you? I really liked the result of your project, but I have some tips that I think you will enjoy:
To make it easier to make the project responsive, use max-width
rather than width
.
To improve the structure of the html document, prefer to wrap all the content with the main
tag, since this challenge is just a component, it needs a main
tag to know what the main content of the page is!
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="catcher">Improve your front-end skills by building projects</h1>
The rest is great!
I hope it helps... π
Marked as helpful
@rezajaber
Posted
@AdrianoEscarabote Hey Adriano, I am fine :) I hope that you are too! Thank you for your help, this was some great advice π
Just yesterday I was talking with a friend of mine about you and how nice your work is, so thank you for taking your time. I really appreciate itπ
@AdrianoEscarabote
Posted
@rezajaber thanks! π happy coding!!