Design comparison
SolutionDesign
Solution retrospective
Hello, feel free to leave your comments 😊
Community feedback
- @pikapikamartPosted about 3 years ago
Hey, awesome work on this one. Layout in desktop looks great and the mobile layout is great as well.
Some suggestions would be:
- The text:
Reliable, efficient delivery Powered by Technology
should only be using one heading tag, an
h1
since those text is just one phrase, the below text is just being wrapped.- The
alt
for the icons should only be usingalt=""
since is is just decorative. If an image only acts as decoration, usealt=""
on it as well asaria-hidden="true"
, but if the image adds content to the webpage, then use a meaningfulalt
value.
Just those two, again, great job on this one.
Marked as helpful0@AnnaUlianovaPosted about 3 years ago@pikamart Thank you a lot for your review and helpful suggestions!
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