Skip to content
Submitted almost 4 years ago

Mobile First Page with Flexbox, Grid, and Vanilla js

Michelle•20
@michellewongi
A solution to the Loopstudios landing page challenge
Preview (opens in new tab)Code (opens in new tab)

Solution retrospective

Hi I would love feedback on if my code is messy and any tips for improve writing clean code...

I also have a technical question regarding how to make the width of the underline that pops out when my mouse hovers over the menu links shorter?

Code
Couldn’t fetch repository

Please log in to post a comment

Log in

Community feedback

  • Yashasvi Singh•890
    @aUnicornDev

    You have border-bottom for the underline so it will take the complete width of the text. For variable widths, you can use ::after pseudo element.

    You can also change .hamburger position from fixed to absolute, because the fixed hamburger appears even after scroll.

    You should set a fixed max-width on the .container so that it does cover the whole screen on bigger viewports.

    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