Skip to content
Submitted over 4 years ago

Responsive landing page with HTML & CSS, toggle JS

Mladen•410
@mladendjokovic
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

I am learning to code, it's my 25th day and I'd appreciate some feedback. Constructive criticism, please.

Community feedback

  • Diarrah•3,418
    @DiarrahPosted over 4 years ago

    Really good job on this!

    My only constructive advice on this would be:

    1. Make sure to add cursor:pointer to all your buttons so that the call to action for the user is super obvious.

    2. Add overflow-x:hidden on the body because in mobile you can scroll to the right & see the mobile nav waiting there. Also, have the mobile nav take up the full 100vw.

    2
    Mladen•410
    @mladendjokovicPosted over 4 years ago

    @Diarrah Thank you for the feedback, it's very helpful! I'll continue learning :)

    1

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