Raymart Pamplona• 16,120
@pikapikamart
Posted
Hey, great work on this one. Both layout in desktop and mobile is good and the responsiveness I think is fine as well.
Some suggestions would be:
- Have a
mainelement that will wrap the whole content. Having amainelement makes your website more structured and this will help also other users to navigate properly on a website. On this one it could be:
<body>
# header is here if there is header for navigations
<main>
All the content in here
</main>
# footer is here if there is a footer
</body>
- You also forgot to add the two round background images on your solution. You might want to add that.
- The text for the ratings beside the stars could have used a
font-weightsetting to a bold like the original. - On the testimonial section, I think you may have swapped the name of the first lady and the guy on the middle. Colton is guy, Irene is lady.
- Also on the
imgof those in the testimonial section. You could have use their name as the value for thealtattribute, likealt="Colton Smith". - The name of the person on the testimonial is right to be a heading tag, but your pick of heading tag is wrong. Do not jump from using 2 level heading tags. What I mean is that, if you use
h1do not jump toh3without havingh2. Use heading tags incrementally by one level.h1toh2toh3without skipping any heading level with 2. - On the mobile layout, I see that you are centering the boxes that holds the rating. Use a
margin: 0 autoso that it will be dynamically centered or just change the0on the margin to adjust the top and bottom margin. Removing also theleftproperty on those boxes on mobile layout would be good, since themarginwill handle the centering of it.
Aside from those, you did a great job on this one.
Marked as helpful
1
faiyaz rahman• 260
@faiyaz-rahman13
Posted
@pikamart thank you so much for helping me and pointing out my mistakes. It means a lot to me and I am very thankful to you for helping me refining and constructing my work more firmly.
0
