Design comparison
SolutionDesign
Solution retrospective
what can you say about this design
Community feedback
- @pikapikamartPosted over 3 years ago
Hey, good work on this one. Layout in desktop is good and the mobile state as well. The responsive does work, but at point between 1130px - 767px, the 3 rectangles that holds the ratings starts to overlap the heading text. You might want to check that one out.
Also, some other suggestions:
- always include a
main
element in a website. On this one, it will include all the elements.heading
main
are some landmarks that uses screen reader to better describe a content. - the
alt
text of the images on the testimonial could be the name of the person, like:alt="colton smith"
. - last, just the mentioned at the start, the overlapping of elements :>
Aside from those, you did a good job on this.
Marked as helpful0@Adegboyega95Posted over 3 years ago@pikamart Thank you very much. I appreciate you for giving me a reply
0 - always include a
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