
Please log in to post a comment
Log in with GitHubCommunity feedback
- P@JiaHe35354
Hello, you're doing great! I have a few suggestions:
In the media query @media screen and (max-width: 1260px), consider using the em unit instead of px. Using em helps keep the layout proportionally consistent because em units are relative to the browser’s default font size (typically 16px). If a user changes their default font size for accessibility reasons, em-based breakpoints will scale accordingly, whereas px is fixed and doesn’t respect user scaling.
Another point: I think the font size on smaller screens (like 375px width) is too small. You might want to increase the minimum font size for better readability on small devices.
- @khadija99-tech
Just small adjustment in CSS like the font, margin and line height of paragraphs.
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