Design comparison
SolutionDesign
Solution retrospective
Continued development
- Adjust Background Body
- Adjust Container Mobile
Feedback are always welcome :)
Community feedback
- @ApplePieGiraffePosted over 3 years ago
Hello there, Ana Paula! 👋
Good job on this challenge! 👏
I suggest,
- Considering using CSS background images to add and position the background shapes in this challenge, since that'll keep the images out of your markup (which will make your HTML slightly cleaner) and I think it's a little easier to manage the background images, then.
- Perhaps using an <article> tag of the card component and a heading tag for the maybe the name of the card to make your HTML a little more semantic (which is good for things like accessibility and SEO). 😉
- Decreasing the space around things inside the card component just a bit (as in the original design).
Keep coding (and happy coding, too)! 😁
1
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