Responsive landing page using flexbox

Solution retrospective
When I styling a website I always feel like this is not the right way to do this. So please show my mistakes and give me suggestions to become better.
Please log in to post a comment
Log in with GitHubCommunity feedback
- Account deleted
Hi there, that looks quit good and it expands smoothly. Improvements can be made by watching your reports. Your social media links are missing an "aria-label" for accessibility. Like <a href="#" aria-label="Link to our facebook page"> This helps screenreaders and their users navigate your page.
Also you should give your button and your links an :active state with another color and/or effect so a user can see that the button was indeed clicked.
Happy coding :)
Marked as helpful - @hameesha
Thank you very much for the feedback.
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