z-mn• 150
@z-mn
Posted
Good job! The desktop solution looks and works well. One thing that you should be aware of is that you should never skip heading tags - you've used a <h3> tag without there being <h1> and <h2>. This is important for screen readers. You also haven't made the design completely accessible on mobile because the card is too wide - you can set a max-width on it so that it is easier to read. The only other issue is that you are missing the mobile active state. Happy coding! :)
0