@ApplePieGiraffe
Posted
Greetings, Simon Hernandez! π
Incredible work on this challenge! π It's great to see you back with another solution! π I really like the animations you added to your solution! π
A small suggestion I have is to perhaps set 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. π
It would also be good to add some focus styles to some of the interactive elements on the page that do not currently have them (such as the "Request invite" buttons).
And it would be worth taking a look at the responsive issue that @veljkoilic mentioned in the hero section of the site! π
Hope you find this helpful. π
Keep coding (and happy coding, too)! π
Marked as helpful
@simonhernandez
Posted
Hello, @ApplePieGiraffe!
It is great to know about you again!
Thanks so much for your feedback and suggestions. I took them into account and applied them on my solution. I also fixed the responsive issue @veljkoilic mentioned.
Thanks again for your always helpful feedback!
@ApplePieGiraffe
Posted
@simonhernandez
Of course! Happy to help! π