Design comparison
SolutionDesign
Solution retrospective
Can you please review my job and please give your suggestion and what should improve about this website and my knowledge
Community feedback
- @ChamuMutezvaPosted over 3 years ago
Congratulations for taking your first challenge.
- let your alt values be very descriptive to help users who rely on assistive technology visualize the message that is being put across.
- headings should be used sparingly. You did well to include an
h1
- is is always required in a website and only oneh1
in a site is considered best practice. Headings should ascend in order - h1, h2 ,h3 etc. The general use of a heading is to give a brief description of what the content of page/section is all about. For that reason in my opinion the headings shown below should have been an element such as span not h3 or h5
<div class="companies3"> <h3>12M+</h3> <h5>queries</h5> </div>
Happy coding
Marked as helpful1 - @AzizkhujaPosted over 3 years ago
Thank you for your suggestion and feedback and happy coding dude
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