Skip to content
Submitted over 2 years ago

Landing page(flexbox)

Abrosss•440
@Abrosss
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

My biggest challenge here was that round underline in links and navigation bar on mobile:D The first time using javascript too.

Community feedback

  • Ollie•580
    @ohermans1Posted over 2 years ago

    Great work on the JS! Very tidy for a first attempt.

    Out of interest, is the white space around the page a design choice? I notice that it goes away for mobile view, but is there for desktop and the screenshot?

    Nice job overall!

    Ollie

    1
    Abrosss•440
    @AbrosssPosted over 2 years ago

    @ohermans1 thank you! ohh i was confused about the width of the page and i tried to narrow it, but i see now i didn't have to do it:D

    1
    Ollie•580
    @ohermans1Posted over 2 years ago

    @Abrosss Looks like a fix is as simple as getting rid of the margin on body :)

    1
    Abrosss•440
    @AbrosssPosted over 2 years ago

    @ohermans1 right i'm going to fix it! and also the size of some elements:D thank you

    1
  • Atta•0
    @attajnr2001Posted over 2 years ago

    I just signed up and it is amazing. U guys are great

    1
    Abrosss•440
    @AbrosssPosted over 2 years ago

    @attajnr2001 Thank you:D and welcome! I hope you will enjoy practicing here!

    0
  • Hassan Yahya•250
    @hassanyahya400Posted over 2 years ago

    The graphics design background image should have backgroun-size:cover and background-position:center center

    1
    Abrosss•440
    @AbrosssPosted over 2 years ago

    @hsn-ng Thank you so much! I just fixed it:D

    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