
Solution retrospective
none
What challenges did you encounter, and how did you overcome them?none
What specific areas of your project would you like help with?none
Please log in to post a comment
Log in with GitHubCommunity feedback
- @lij110397
**It does not include semantic HTML. <main> label is not included in the <body>, which makes the code more understandable. ** The overall design is correct and neat. However, the screen should be scrollable instead of placing all in the current viewport.
- The code in the body should be put in the <main> </main>.
- The height of <main> should adjust to the content.
I like how you put all global font setting in the :root variables
--font-size: 16px; --font-family-young-serif: 'Young Serif', serif; --font-family-outfit: 'Outfit', sans-serif; --font-weight-regualr: 400; --font-weight-bold: 600; --font-weight-extra-bold: 700;
Marked as helpful
Join 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