@FluffyKas
Posted
Hey, great solution! You even added your readme, very nice :) Something that perhaps you could improve is your html. Switch your container div
to main
, that gets rid of the issue in the accessibility report. The other thing is your image alts. They have a specific role to fill, I'd suggest reading this, it was a game-changer for me: (https://axesslab.com/alt-texts/)
In this challenge, I believe they can be just left empty, but I still suggest reading this great article for future projects! :)
Marked as helpful
@syzwnftr
Posted
@FluffyKas Hi, thanks for your feedback. And yeah, you're right! I need to improve my html. I've been dealing with the accessibility issue since my first challenge and I just turn a blind eye on it. Perhaps this is the right time to learn more about accessibility, and apply it on my next challenge. I'll be sure to check out those link that you suggested. Cheers! :)
@FluffyKas
Posted
@syzwnftr Accessibility is a big and complicated topic. I'm still learning it myself but taking it step by step helps ^^ After all, you can always return to older projects and correct them a bit :) Good luck!