Responsive Social Proof Section using Flexbox & SASS/SCSS
Design comparison
Solution retrospective
Hello! For this challenge, I used Flexbox and SCSS (mobile first). Any feedback are greatly appreciated! Thank you!
Community feedback
- @pikapikamartPosted almost 3 years ago
Hey, really nice work on this one. The desktop layout looks fine, though it needed that two background-blobs and the
background-color
for each of the review so that there will be contrast. The site is responsive thought I still find it large to use 0px-1050px just for mobile breakpoint since desktop layout could use more of those screen-time.Here are some suggestions for the site:
- Adding a
max-width
on thebody
would be really nice. If you zoom out on your screen, you will see that the layout just stretches along the screen since there nomax-width
that will limit the site. Adding that would be nice. - Adding the
background-color
on the container for each of the review. - For those stars in the review, you could use the
background
property to set multiple star-icons sincebackground
can accept multiple images. This way, you won't have to create extraimg
tags on the markup. But if you insist on usingimg
tag, then add an extraaria-hidden="true"
on each of the star-icons so that they will be skipped by screen-reader. - You can change the
section
tag that wraps the testimonial part into justdiv
sincesection
is not informative as landmark unless it is labelled usingaria-labelledy
or it can be used if there is already a visible heading text on the layout. - For each of the testimonial card, you can use this markup below so that it will be easier for screen-reader users to traverse it using
blockquote
:
<figure> <img src="" alt={person name}> <blockquote> <p> {qoute in here}</p> </blockquote> <figcaption> person name <p> person role </p> </figcaption> </figure>
You will just need to use
display: grid
on thefigure
so that you could place each item properly like on the design.Aside from those, great job again on this one.
Marked as helpful2 - Adding a
- @BriuwuPosted almost 3 years ago
I see! Thank you so much for the feedback! I will try doing these suggestions! Thank you!
19
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