Samuel Palacios• 615
@samuelpalaciosdev
Posted
Hello, Spymom! 👋
Well done on this challenge!. Your solution looks great and it scales pretty well.
I only suggest a little thing😉:
- I think that those images, don't need an alt text. In this instance, that's a good thing, as the alt text is currently repeating the same as the card headings, so if you take this tip, use
alt="" aria-hidden="true"on your HTML markup.
I really like the way you approach this, and that fancy animations you added are so cool, keep coding ;)
1
spymon• 90
@spymon
Posted
Hi, @samuelpalaciosdev 👋
Thank you for pointing that out! Now I know something new 😊
I did a little research about what alt="" aria-hidden="true" does and implement it in my code.
Thank you! 😉
0
