Mobile first HTML + CSS using BEM and media queries - 375px and 1440px

Please log in to post a comment
Log in with GitHubCommunity feedback
- @grace-snow
Hi
The main question here is why add a hover effect to the cards at all? A hover is always to indicate interactivity so should only be used on interactive elements.
Other feedback is
- icons are spilling out of the cards for me on mobile, test on more screen sizes, including landscape
- font size must never be in px, always a relative unit
- if you want to use BEM naming research more, especially how to indicate modifier classes vs child classes
Good luck
- @LeTailleCrayon
Hello, The design and the solution don't match !
Join our Discord community
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord