@pikapikamart
Posted
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 helpful
@inanbruneli
Posted
@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!