Design comparison
SolutionDesign
Community feedback
- @Yakub-EgamnazarovPosted almost 3 years ago
First of all: congrats, the layout works well in both device widths. However, in order to solve some accessibility issues, you could use <main> element instead of div, in your code it is <div> with "all-cards" class. You can follow the frontend validator to solve those issues. Secondly, in order to place all your grid in the middle, you could apply flexbox centration (e.g. align-items and justify-content) for the <div> element with "all-cards" class. The main container.
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