Design comparison
Solution retrospective
I am current working in improve my CSS skills, so I would be happy to hear what I could do different/better than this. So, feel free to put your feedback ๐๐
Community feedback
- @pikapikamartPosted about 3 years ago
Hey, awesome work on this one. Though I had to zoom out to see the desktop layout since you are using
media (min-width: 1440px)
, for the desktop layout, this is too big for users like me who uses a traditional 1366x768 resolution. Lowering it down would be really great, though the mobile layout looks good.Some suggestions would be:
- Always have a
main
element on a webpage, this will wrap the main content of the page, for this one, you should use amain
element after thebody
tag, then the rest of the elements are inside themain
. - The text after the
h1
should just onlyp
tag and noth3
. - The rating text as well shouldn't be a heading tag, just a regular
p
tag would be better. - On the
body
tag, do not useoverflow: hidden
, try inspecting your layout in dev tools at the bottom, you will notice that you can't scroll, it is because of that styling. - The
alt
values for each of the testimonial images should be using their names likealt="Colton Smith"
. - The name of each person should be using a heading tag like
h2
. Also, do not just use a singlediv
tag to wrap a text, usep
tag on them. Always wrap content in a meaningful element. - The testimonial content text shouldn't be a heading, just use
p
tag on is, since it is just a regular text/testimonial.
Aside from those, great work again on this one.
Marked as helpful1@inanbruneliPosted about 3 years ago@pikamart I really appreciate you for taking time for helping me!! All the information you gave me war helfull, and I will apply the changes on my project and think better on the next one. Thank you again!
1 - Always have a
- @BurritoDoggiePosted about 3 years ago
Hi!
Great job on the challenge! I can tell you a lot of effort into it, and you did just fine on your CSS! I usually give feedback on a mobile device and your project fit perfectly! If you want you can look at the 'report' above to help with your code. But overall I think you did wonderful!
Keep Coding!
(โโกโโก)
1
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