Design comparison
SolutionDesign
Community feedback
- @grace-snowPosted almost 3 years ago
Good job overall
You need to make figcaption the first child of figure element (it's invalid html at the moment). It's fine for the figcaption to include an image so it's an easy change to make - swap div and figcaptions
Dont forget to add the box shadows like the design.
Some of the text is unreadably small for me. That may be due to the design but if a designer asked me to build text that small I'd challenge it. It's fine to make changes to make solutions more accessible where necessary
Nice solution overall though, well done
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