@ApplePieGiraffe
Posted
Hello, Henry Andrew Baum! 👋
Well done on this challenge! 👍
If your CSS file is becoming very long and harder to navigate, it may be worth looking into using a tool like Sass to make writing and managing your styles easier. Using Sass you can split your styles into smaller files for organization and do cool things like write simple "functions" to avoid repeating code.
It's a good practice to use ems/rems over pixels for font sizes for accessibility reasons. Here's a great article on the topic if you're interested in learning more about it. I'd also recommend using ems/rems for other the values of other properties in your CSS such as margin, padding, etc., as that will cause the rest of the design of your site to scale with the user's chosen default font size. 🙂
As for your solution, I'd suggest leaving the alt text for the icons in the card component empty since they are mostly decorative and don't need to be read by screen readers.
And don't forget to use a button element for the button in the card. 😉
Hope you find this helpful. 😊
Keep coding (and happy coding, too)! 😁
Marked as helpful
