CSS Flexbox

Solution retrospective
I will be happy, to hear any feedback and suggestion
Please log in to post a comment
Log in with GitHubCommunity feedback
- @boedegoat
Great work AditNovadianto. You've done fantastic job
In order to fix your accessibility issue, you can add
<main>
tag to be the wrapper of your entire component.Other than that, stay code and happy coding
Marked as helpful - @moheb2000
Awesome 👏👏👏
1- For 'Why Us' section it's better to use lists in HTML because it actually is a list but if you don't want to use that you need to put them in different lines.
2- don't use flex method for centering element in mobile view because it's cause overflow issues. You fixed overflow in vertical direction but in horizontal direction some texts aren't readable.
Great work. Good lock ❤️❤️❤️
Marked as helpful
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