Skip to content
Submitted about 2 years ago

Responding landing page using flexbox

Mennatallah Hesham•1,490
@Mennatallah-Hisham
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Insure landing page
  • HTML
  • CSS
  • JS
2junior

Design comparison

SolutionDesign

Solution retrospective

what improvements could be made? did I use semantic HTML correctly? is my code well-structured?

Community feedback

  • Divine Obeten•2,415
    @Deevyn9Posted about 2 years ago

    Hi, congrats on completing this solution, your solution looks really great. However, I have a few feedbacks.

    Firstly, there is no horizontal padding in the mobile navbar, the burger button and the company logo have no gap with the edges.

    Secondly, You'll want to remove the scroll whenever you toggle the navbar on mobile, you can do this by adding an overflow of hidden on opening the navbar and removing the hidden overflow on closing the navbar.

    I hope you find this really helpful.

    Happy coding!

    Marked as helpful
    0

Please log in to post a comment

Log in with GitHub

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