Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Responsive landing page using flexbox

@hameesha


Design comparison


SolutionDesign

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.

Community 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

1

@hameesha

Posted

Thank you very much for the feedback.

0

Please log in to post a comment

Log in with GitHub
Discord logo

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