Design comparison
Solution retrospective
Hi! I will be thankful for any comments, advice and feedbacks :-)
Community feedback
- @grace-snowPosted about 4 years ago
This looks really nice code Joanna.
On your grid, remember you can use shorthand for css values like
gap: 25px 25px;
- no need to have two values if your row and col gap are the same 😊And on your testimonial-header, you could make that a
header
html element if you wanted. Not necessary, but would make sense.The only thing I question in your html markup is the h2s. They don't feel like they should be headings to me. If you imagine the whole document content written out in plain text, what would be the heading? I expect it would be the person's name....
But really these are down to personal choice I think.
Good job
1 - @maxenceyrowahPosted about 4 years ago
I find your work super impressive Joanna Skrzypczak but I have a little problem when I open the chrome console to test the responsive there is a small bug in the centering
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