Responsive Layout, flexbox, css, @media queries
Design comparison
Solution retrospective
Review my project please 🙏.
If you want to suggest anything, please go ahead.
Community feedback
- @Illyaas4ShowPosted over 2 years ago
This looks like a great attempt, well done! A few things needs to be adjusted though...
-
Heading levels should only increase by one
- Your page must have a level one heading (<h1>
) and the go down one by one every time. Your next header should have a<h2>
not a<h3>
. It gives your page a good structure and helps screen readers navigate through the page. -
And for the HTML issues, I don't thing there is a need to add a
<div>
or a<p>
tag inside the button as it is causing some problems.
I hope this helps. If it does please mark as helpful thx Happy coding :)
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