Responsive four card feature page

Please log in to post a comment
Log in with GitHubCommunity feedback
- @Chanokthorn
- Does the solution include semantic HTML?
- Proper usage of elements.
- Is it accessible, and what improvements could be made?
- The code can benefit from including Landmarks, see ARIA suggestions on the result when submitted.
- Does the layout look good on a range of screen sizes?
- It does, with beautiful twist on two-columns mid screen-transition. Great work!
- Is the code well-structured, readable, and reusable?
- Readable structure, class-names, and comments.
- Does the solution differ considerably from the design?
- Matches the design, no problem here at all.
Additional notes
- You might benefit from setting font-size: 15px on html {...} level of css, and use 'rem' units instead of 'px'. It seems to improve scalability of the design, from what I understand.
- Does the solution include semantic HTML?
Join 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