responsive page

Please log in to post a comment
Log in with GitHubCommunity feedback
- @Oreolis
Hi, nice job, generally. But if I may add some hints :) HTML: -- add alt="" to the images, also try adding loading="lazy" for better performance. -- <p class="Perfume"> don't do class naming with capital letters -- consider using <main>, <article> etc for better semantic
CSS: -- pay attention to details -- try adding CSS custom attributes and then load them -- clean your code, like this part here: @media max- {
} -- avoid using px, try rem/em instead.
Other than that .. good job and happy coding! :)
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