Skip to content
Submitted over 3 years ago

Using flexbox

Code-decode•145
@sayedlee
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Project tracking intro component
  • HTML
  • CSS
  • JS
2junior

Design comparison

SolutionDesign

Solution retrospective

any suggestion? how can i improve more. if anybody did this project i want to know ,im having problem on positioning background image in mobile view.

Community feedback

  • Raymart Pamplona•16,100
    @pikapikamartPosted over 3 years ago

    Hey good work on this one. The layout in desktop is good.

    Regarding your concern in the mobile view. Well I see that you declared a width: 100% for the image right, but you declared it at the wrong element, you should have declared that in the img itself, not the container holding that image. That will not work. Changing that will be really good so that you can achieve what you are looking okay.

    Also your navbar dropdown is a bit off when it is popped out. I suggest adding width: 100 to it and removing the right property that is declared on it. This way, it will take the size based on the header and it will be properly placed centered.

    Overall, you did good job in here^^

    1
    Code-decode•145
    @sayedleePosted over 3 years ago

    @pikamart thanks for your suggestion ,i have fixed my nav menu hope this works now.😄

    0
    Raymart Pamplona•16,100
    @pikapikamartPosted over 3 years ago

    @sayedlee Awesome^^

    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