@ApplePieGiraffe
Posted
Greetings, Mojtaba! 👋
Good job on this challenge! 👏 Overall, your solution looks pretty good and responds quite nicely! 👍
A few things I'd like to suggest are,
- Centering the content of the page on extra-large screens (at the moment, it sticks to the left of the page when the width of the screen increase in the desktop layout).
- Not adding
alt
text to the star icons if you use<img>
tags to add them to the site (since screen readers will read out thealt
text for every single star icon, then, which can be annoying). If you like, you can use CSS background images to add the star icons (play around with thewidth
of the container and thebackground-repeat
property to make the star icons repeat at the proper number).
Hope those tips help. 🙂
Your HTML looks good! 🙌
Keep coding (and happy coding, too)! 😁
@MojtabaMosavi
Posted
@ApplePieGiraffe thank a lot for the great tips.
@ApplePieGiraffe
Posted
@MojtabaMosavi
No problem! 😀
Oh, and BTW, if you found those tips helpful, an upvote would be appreciated! ;)