Design comparison
Community feedback
- @JawadM2002Posted about 2 months ago
This seems like a great effort, and you gave it your best shot. There is a meaningful use of semantic HTML with descriptive classes to help define the different parts of the component. The solution itself is accessible, and what you can improve on it is that you can add a label element with the button element to improve accessibility for screen readers. Moreover, the layout looks really good on a range of screen sizes, and you used relative units to make it more responsive, as well as a media query of 600 pixels to adjust the layout for larger screen sizes. The code is well-structure, readable and reusable, as there are consistent naming conventions and clear class names. The only difference in the solution is the size of the card, but the position of the elements inside remain the same. Overall, well done and keep it up. :)
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