Adriano• 34,090
@AdrianoEscarabote
Posted
Hi Mohamed Nejjar, how are you?
I really liked the result of your project, but I have some tips that I think you will enjoy:
To centralize your content, prefer to do this:
body {
margin: 0;
padding: 0;
display: flex;
align-items: center;
justify-content: center;
min height: 100vh;
}
The document should have one main landmark, to solve this accessibility problem, wrap all the content with the main tag, since this project is based on just one section!
The rest is great!
I hope it helps... 👍
Marked as helpful
0
Mohamed Nejjar• 170
@NJR911
Posted
@AdrianoEscarabote Hi! thank you for your feedback that's really helpful, i'll try to do that next time.
0
