@ApplePieGiraffe
Posted
Hello there, Ammar Jalabi! π
Nice work on this challenge! π Your solution looks great and responds nicely! π
A couple of suggestions I have are,
- Wrapping each of the social media stats cards in a link tag (since it seems like they would take the user somewhere that displays more information when they are clicked). Right now, those elements are just plain
div
s (which works, but isn't great for accessibility or semantics). - Adding a visible focused state to the theme toggle (so that keyboard users can tell when that element is highlighted). Not adding focused states to interactive elements can make your page difficult to navigate for keyboard users. π
Hope you find this helpful. π
Keep coding (and happy coding, too)! π
Marked as helpful
@AmmarCode
Posted
@ApplePieGiraffe Thank you so much for your great tipsπ I will update my code soon and would keep the keyboard users in mind from now on. π Happy coding! π