Design comparison
Solution retrospective
Open for suggestions...
Community feedback
- @RioCantrePosted over 2 years ago
Hello there! Awesome work in this challenge. Viewing at your solution, I would like to recommend the following for you...
- In the
.container
, Removemargin-left: auto;
andmargin-right: auto;
change intomargin: 0 auto;
. Same goes withpadding-left: 0.75rem;
andpadding-right: 0.75rem;
change it intopadding: 0 0.75rem;
. Apply the same in the media query
Above all, the design in looking great as well as the responsive approach. Hope this helps and Keep it going!
Marked as helpful1@RioCantrePosted over 2 years ago@Kajaia No worries! I would appreciate also if you mark this as helpful! Cheers!
0 - In the
- @NaveenGumastePosted over 2 years ago
Hay ! Lasha Good Job on challenge
These below mentioned tricks will help you remove any Accessibility Issues
-> Add Main tag after body
<main class="container"></main>
-> Learn more on accessibility issues
If this comment helps you then pls mark it as helpful!
Have a good day and keep coding 👍!
0@KajaiaPosted over 2 years ago@Crazimonk As I remember main tag is after body in this project
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