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
mainelement on a webpage, this will wrap the main content of the page, for this one, you should use a
mainelement after the
bodytag, then the rest of the elements are inside the
- The text after the
h1should just only
ptag and not
- The rating text as well shouldn't be a heading tag, just a regular
ptag would be better.
- On the
bodytag, do not use
overflow: 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.
altvalues for each of the testimonial images should be using their names like
- The name of each person should be using a heading tag like
h2. Also, do not just use a single
divtag to wrap a text, use
ptag on them. Always wrap content in a meaningful element.
- The testimonial content text shouldn't be a heading, just use
ptag on is, since it is just a regular text/testimonial.
Aside from those, great work again on this one.
Marked as helpful
@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!