@ApplePieGiraffe
Posted
Hello, Zdravko! ๐
Good job on this challenge! ๐
A couple of things I suggest are,
- Setting the
alt
text for the star icons to be an empty string so that they will be ignored by screen readers. Thatโs because those elements aren't so necessary or important to the content of the page and as a result doesnโt need to be read by screen readers (which can be annoying for users if screen readers are reading out thealt
text for every single star). ๐ - Adding
overflow-x: hidden
to thebody
to prevent any unwanted horizontal scrolling on the page. - I think adding the decorative background images with CSS background images to the
body
of the page will do (there's no need to add extradiv
s for those images). You can simply add both of them to thebody
of the page. See this helpful article for how to use multiple CSS background images:
Hope you find these suggestions helpful. ๐
Keep coding (and happy coding, too)! ๐
Marked as helpful
@Zdravko93
Posted
Hello @ApplePieGiraffe, and thank you for your helpful suggestions. I will edit my code according to them, and have those tips in mind while building my next project. Thanks again :)