Design comparison
Solution retrospective
Please leave any constructive criticism/tips and feedback!
Community feedback
- @kmeganizPosted almost 3 years ago
Good work. Seeing your report I try to help as I also got help from someone around this:
-
First of all, please put in your Readme file share what your struggle and create the Live URL so easier for us to check.
-
An img element must have an alt attribute, except under certain conditions. For details, consult guidance on providing text alternatives for images. I suggest for all the img put the alt like below
<img src="images/illustration-hero.svg" alt="illustration">
-
Page should contain a level-one heading I suggest your <h3 id = "h3">Order Summary</h3> the h3 can change to h1 and apply the style accordingly.
-
All page content should be contained by landmarks In the body you can add <main> then <footer>
That's for now. Hopefully this is helpful.
Marked as helpful0@GamuchiraiSPosted almost 3 years ago@kmeganiz Thank you so much for the feedback! It is extremely helpful
0@kmeganizPosted almost 3 years ago@GamuchiraiS Hi thanks for replied. If you think it is helpful, can you mark as helpful? Thank you
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