Faris P• 2,810
@FarisPalayi
Posted
Looks nice and responsive 👍
A few suggestions are:
- Set the image's alt tag to
alt=""since the image here doesn't provide any extra context or info to users. And so that assistive devices can ignore them. - Try using a smaller media query breakpoint because, even though currently I'm using my laptop to see your solution, I'm getting the mobile version of the site.
- Also, on mobile, the attribution text is on top of the card, so try to change it if you can😊
That's all from me. Have fun coding ✨
Marked as helpful
1
leonardo reis• 250
@leonardo9245
Posted
@FarisPalayi Hi, thanks for your feedback, I made the changes, could you check again?
thankful
1
Faris P• 2,810
@FarisPalayi
Posted
@leonardo9245 Looks great👍 and layout is responding really well. Well done 👏
btw, the font-family of the text under stats(for eg: "companies") is lexend deca not Inter😀
One more suggestion would be to change those h2 tags to <strong> or <span> because those are technically not heading for anything. Take a look at Semantic HTML
As always, have fun coding 🌟
1
