Hi Luka
This looks fantastic! Just a few improvements needed on the html
Those buttons should be anchor tags as they would trigger navigation, not an action
It's up to you if you think those icons are meaningful, in which case you should include a value in the alt attributes. If you think they are not meaningful/important content, leave the alt empty like alt=""
. Or if you think they are valuable content, the alt text needs to be written slightly differently to what you have now. You don't usually say words like image or picture because they are announced as images anyway, but the word icon -could be helpful. If words are initials or abbreviations like SUV they need to be capitalised for screen readers to read them correctly.
Really nice looking solution. I hope this extra info is helpful for you in this or other challenges
@Luka85
Posted
@grace-snow I fixed the issues, so changed the button to anchor tags and change the alt attributes to alt="" cause they are not meaningful. What do you think now?
Thank you for your feedback,
Luka