Responsive Flex box using HTML and CSS
Design comparison
Solution retrospective
All feed back are welcome
Community feedback
- @HassiaiPosted almost 2 years ago
There is no need for <section>, replace <div class="container"> with the main tag in the html this will fix the accessibility issue. In the css replace the replace section with the body, there is no need to give the body a width value and flex-direction: column;
Use the color that was given in the styleguide.md found in the starter folder; give .p a color value like the one in the design
Use rem or em as unit for the padding , margin, width and preferably rem for the font-size for more on this watch this https://youtu.be/N5wpD9Ov_To
Hope am helpful HAPPY CODING
Marked as helpful1@sulyman09Posted almost 2 years ago@Hassiai Noted Thanks Boss My question is that if I want to fix the accessibility issue is it the GitHub am to apply the corrections? And after applying the Correction do I still need to submit on frontend again. Pls I am a novice
0@HassiaiPosted almost 2 years ago@sulyman09 you can do that in the github , the changes will reflect wait for 20 mins approximate to see if the effects. Well done for completing this challenge
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