Design comparison
Solution retrospective
Second Challenge accept! I still need to learn a lot of things like grid-layout and overflow. And sure, I am accepting any advice and comments on my code.
Community feedback
- @hitmorecodePosted over 1 year ago
You forgot to add the hover effect on the buttons and your page is not responsive. You can use media query to do so. For the coming challenges, try to go with mobile first approach.
@media only screen and (max-width: 700px) { .cards { grid-template-columns: 1fr; grid-template-rows: repeat(3, 1fr); } }
Marked as helpful0@rowanDeveloperPosted over 1 year ago@hitmorecode you are right. I forgot about the hover-effect and I didn't made it responsive (most likely because I am struggling with media query). But thanks a lot for your advice!
0@hitmorecodePosted over 1 year ago@rowanDeveloper I know the feeling. I used to be afraid of media queries 😅. if you need help regarding media query let me know
Marked as helpful0
Please log in to post a comment
Log in with GitHubJoin 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