Design comparison
SolutionDesign
Community feedback
- @AdrianoEscarabotePosted about 2 years ago
Hi @BekEsaly, how are you?
I really liked the result of the 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 here2- I noticed that the content is not very centralized, I made some adjustments to correct this:
body { /* margin: 0 auto; */ display: flex; flex-direction: column; justify-content: center; }
The rest is really good!
Hope it helps... 👍
Marked as helpful0 - @BekEsalyPosted about 2 years ago
Hi! Thanks a lot for your advice! I coded this and other' on my phone 😉 for practicing! I'll try to correct mistakes...
0
Please log in to post a comment
Log in with GitHubJoin our Discord community
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord