P
ApplePieGiraffeβ’ 30,545
@ApplePieGiraffe
Posted
Hey there, Veljko Ilic! π
Great job on this challenge! π Your solution looks good and is responsive! π
A couple of suggestions I have are,
- Turning the navigation links in the header and footer of the page into actual links by wrapping each of them in a link tag.
- Turning the article cards in the "Latest Articles" section into links as well since they seem as if they would take users to their respective article when clicked.
- Setting the
alt
text for the icons in the "Why choose Easybank?" section to be an empty string so that they will be ignored by screen readers. Thatβs because those elements aren't really necessary or important to the content of the page and as a result probably don't need to be read by screen readers. - Adding some more descriptive
alt
text the images in the cards in the "Latest Articles" section (unless you'd prefer to hide those from screen readers, as well, which should be okay). Thatβs becausealt
text is read by screen readers to describe images to users, so itβs important to writealt
text that makes sense and will give users a good sense of the content and purpose of an image. Currently, thatalt
text is exactly the same as the titles of each of the cards, which might not be the most helpful! π
Hope you find this helpful. π
Keep coding (and happy coding, too)! π
1