Design comparison
SolutionDesign
Solution retrospective
What are you most proud of, and what would you do differently next time?
Managed to get it working pretty well and close to the design. I think my card numbering is off so would make sure that is correct in mobile first before starting desktop
Community feedback
- @py-code314Posted about 1 month ago
Hello,
Good job working on the solution 👏🏼. It looks good on both mobile and desktop screen sizes. Kudos for good use of CSS Grid
As I went through your code I thought I could make some suggestions to improve the code. Here they are:
HTML:
- For
.card
class you can use <figure> instead of <div> if you wanna use semantic HTML tags. Then you can use <figcaption> for.card__user_profile
- You can also wrap testimonial text inside <blockquote> and two <p>s instead of two separate <div>s to make it more accessible
- The images are mostly decorative so you can leave 'alt' empty. I believe it's not a good practice to use 'picture' or 'photo' in alt text
CSS
- You can add more custom variables like 'font-size', 'font-weight' and 'spacing' variables in
:root
if you wish - I suggest to use a global CSS reset either by Andy Bell or Josh Comeau
- Around 600 - 800 px screen width some cards have too much gap between second paragraph and bottom border and I guess it's because of
min-height
you have in.card
- Border around images looks a little thicker in design files. You can probably use 2px there
Don't have too many suggestions 😁 and hopefully these are helpful to you. Wish you all the best!
0 - For
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