Design comparison
Solution retrospective
Overall, the project was easy. A simple question I have is whether I followed the adequate hierarchy in CSS because I feel like it's a mess. Let me know what you think. Thanks again
Community feedback
- @AdrianoEscarabotePosted about 2 years ago
Hi Cousnay, how are you?
I really liked the result of your project, but I have some tips that I think you will enjoy:
To center the content in the center of the screen, we can do this:
body { display: flex; align-items: center; justify-content: center; flex-direction: column; min height: 100vh; }
For the user who uses assistive technologies, such as screen readers, to indicate what the main content of the page is necessary to have the
main
tag, we can solve this problem by involving all the content with themain
tag.The rest is great!
I hope it helps... 👍
Marked as helpful0@Disney-BanjePosted about 2 years ago@AdrianoEscarabote thank you for the review and advice on my project. Highly appreciated. I have a question though why did you use:
body { min-height: 100vh; }
instead of
body { height: 100vh; }
I would like to know why.
Thanks again.
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