Skip to content
Submitted over 3 years ago

First Landing Page, Mobile first Workflow, CSS Grid, CSS Flexbox

Gilang Aditya•425
@madegilangaditya
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Agency landing page
  • HTML
  • CSS
  • JS
2junior

Design comparison

SolutionDesign

Solution retrospective

I have finished my first landing page challenge. Every feedback will be very appreciate, especially for code optimization. Thanks

Community feedback

  • Talha•120
    @TalhaAmjad0034Posted over 3 years ago

    Damn looks perfect to me!! Change icon to hamburger menu to cross or close on click... Keep going!! Great work

    0
    Gilang Aditya•425
    @madegilangadityaPosted over 3 years ago

    @arywahhh Thanks Man

    0
  • ApplePieGiraffe•30,545
    @ApplePieGiraffePosted over 3 years ago

    Greetings, Gilang Aditya! 👋

    Nice job on this challenge! 👏 Your solution looks good and scales up/down very well! 😀

    The only minor thing I suggest is to perhaps add a "close" icon for the mobile menu that gets displayed when the menu is open (since currently, the icon for the mobile menu simply remains the same). 😉

    Keep coding (and happy coding, too)! 😁

    0
    Gilang Aditya•425
    @madegilangadityaPosted over 3 years ago

    @ApplePieGiraffe thanks man, will do the close icon on menu

    0
  • Marko•65
    @remarkovic98Posted over 3 years ago

    Pretty cool :) Just keep working

    0
    Gilang Aditya•425
    @madegilangadityaPosted over 3 years ago

    @remarkovic98 thanks man

    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