
Please log in to post a comment
Log in with GitHubCommunity feedback
- @NIROTRINLAND
Great job overall! Your solution shows a solid grasp of semantic HTML, responsive layout, and clean, organized CSS. You’ve used meaningful tags like
<article>
, lists, and headings well, and the responsive design with media queries works nicely on different screen sizes. The font choices and color scheme are pleasant and accessible, and your code is readable and structured. However, there are a few improvements you could make: the imagealt
text should be more descriptive for accessibility, and using semantic elements like<main>
and<section>
would help structure the page better for screen readers. Also, the CSS selector.container div:not(first-child)
is invalid and should be replaced with a class-based approach for better targeting. Minor layout issues on smaller screens could be fixed with additional padding. Overall, it’s a strong submission with just a few tweaks needed to make it excellent. Keep it up!
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