T
ApplePieGiraffe• 30,545
@ApplePieGiraffe
Posted
Hello, Andrija! 👋
Well done on this challenge! 👍 Your solution looks great and responds nicely! 👏
Just one or two things I suggest are,
- Using a
button
element for the mobile menu button in the mobile view of the site in order to make your solution more accessible. - Setting the
alt
text for the social media icons in the footer of the page to an empty string so that they are ignored by screen readers (since they aren't necessarily important to the content of the page). It would be better to add some screen reader-only text inside those links or add an `aria-label` attribute to them instead to make sure they can be identified by screen readers.
Hope this helps. 😊
Keep coding (and happy coding, too)! 😁
Marked as helpful
2