Design comparison
SolutionDesign
Solution retrospective
any feedbacks appreciated thanks!
Community feedback
- @ChamuMutezvaPosted almost 4 years ago
Welldone Kucabara21. Here are some few suggestions on things that i noted:
- For some medium devices , there is a large whitespace at the top which is caused by the justify-content: center on the body. check at screen width of about 935.
- Heading elements are supposed to be in order. The first level heading is supposed to be h1 followed by h2 and so on. This is important for accessibility among other things. For further reading see link [https://developer.mozilla.org/en-US/docs/Web/HTML/Element/Heading_Elements] If you find this helpful please upvote me Happy coding
2@Raihan-21Posted almost 4 years ago@ChamuMutezva oh wow thanks for the heading info so i should've just edit the font size? i cant find any whitespace at 935 width screen? I already upvoted ur commment thanks
0@ChamuMutezvaPosted almost 4 years ago@Kucabara21. Yes - use css to style the font to any required size. Let me see if I can post an image of the screenshot.
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