@AdrianoEscarabote
Posted
Hi @Durgathev, how are you?
Welcome to the front-end mentor community. 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 p.heading
to h1
click here
I noticed that you used position: absolute;
to center the content, always prioritize using flex-box
or grid
body { display: flex; align-items: center; flex-direction: column; justify-content: center; }
.main { /* position: absolute; */ /* top: 50%; */ /* left: 50%; */ /* transform: translate(-50%, -50%); */ }
Commented codes have been removed! Hope it helps... π
Marked as helpful
@Durgathev
Posted
Hey @AdrianoEscarabote, I'm doing good. thanks. Hope you are doing well.
Thanks for your valuable feedback. It's worthwhile and I'll practice it in future projects.
I'd love to hear you again! π
@AdrianoEscarabote
Posted
@Durgathev I'm glad you liked it! keep coding! π