Design comparison
SolutionDesign
Solution retrospective
Hi, Kindly comment what do you think about my submitted solution, any helpful tips will be much appreciated. Thank you!
Community feedback
- @ChamuMutezvaPosted almost 3 years ago
Greetings Quincy. Here are some suggestions
- your project is not responsive. The site has to viewed properly on almost all devices, i cannot view it on mobile for instance.
- the following meta tag is missing in the head section
<meta name="viewport" content="width=device-width, initial-scale=1" />
. That is the tag that makes your site responsive, it tells the browser to set the width of the content to the width of the device itself, and to scale that content to 1 on load. - images must have alt values, decorative images can have alt="" while the rest should have an alt value that is beneficial to screen readers by being descriptive.
- the h1 heading element should be used sparingly. For best practice , there should be one h1 element per site with the headings following a sequential order. The first heading being the h1.
- the faqs, contact, privacy - most of the footer elements should have been anchor elements , those are links to the related pages
- a span being an inline element should not be a parent to a block element
- generate a new screenshot so that the current solution screenshot can be removed
- go over the other issues raised in the report as well
Marked as helpful0@quigenPosted almost 3 years ago@ChamuMutezva Hi Chamu, I'll try your advice. Thanks for taking time and sharing these helpful tips.
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