Design comparison
Solution retrospective
Hey guys! Any feedback is welcome!
Community feedback
- @grace-snowPosted over 3 years ago
This looks good.
Just a small issue with the headings. They have to go in order. But I think that first heading split over 2 lines should just be one heading with how it reads.
I'd also recommend you have another pass at this using css grid instead of flexbox. It's the perfect challenge for grid and will use need less html structural markup
0@LesleyWesleyPosted over 3 years ago@grace-snow Thanks for the heads-up about the headings! I fixed them and they should be correct now. I just flipped the h1 and h2 tags so I wouldn't have to change all my css.
I actually haven't learned grid yet, but I'm starting an "advanced css" class next week that has it in the curriculum, so after I finish that I may come back to this challenge and give it another go.
0
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