Design comparison
SolutionDesign
Solution retrospective
I am working hard on my CSS skills. Please give me your valuable feedbacks.
Community feedback
- @subhamenginePosted almost 3 years ago
Thanks for the valuable feedback. It's highly appreciable.
0 - @martinelias1312Posted almost 3 years ago
Hello @subhamengine, few tips about accessibility issues:
- main content can be wrapped in
<main>
element<div class="background">
- <h1> should be on page, but used only once per page, article about it [here](https://developer.mozilla.org/en-US/docs/Web/HTML/Element/Heading_Elements)
- this
<div class="attribution">
can be wrapped in<footer>
element alt="image"
alternate text should be more descriptive like: "Women sitting in an office"
Anyways nice job and hope my suggestions help :)
0 - main content can be wrapped in
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