Design comparison
Solution retrospective
Please check it out and rate my work
Community feedback
- @Mayurtm29Posted over 2 years ago
give alt attribute to images and id has to be unique so instead of id use class to target multiple elements at once and background quotation.svg is overflowing correct that.
happy coding
0 - Account deleted
Hello there! 👋
Congratulations on finishing your challenge! 🎉
I have some feedback on this solution:
-
Always Use Semantic HTML instead of
div
like<main>
<header>
<footer>
, etc for more info -
Consider using the h1 element as a top-level heading only (all h1 elements are treated as top-level headings by many screen readers and other tools) like for example use h1 first then h2 etc.
i hope this is helpful and goodluck!
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