Skip to content
Submitted over 2 years ago

Responsive intro section using css grid and flexbox

Priya Nayak•130
@priya180975
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Intro section with dropdown navigation
  • HTML
  • CSS
  • JS
2junior

Design comparison

SolutionDesign

Solution retrospective

Any suggestion to improve would be appreciated.

Community feedback

  • Ben 🌌•570
    @bengeraPosted over 2 years ago

    Hello Priya Nayak.

    Looks good!

    • I think you could wrap your whole page, give it a max-width value and center it so the page content stays together on larger screens. The same for the 'content' and 'hero__img' divs.
    • You could also add a bit of extra JavaScript so the sub menus for the mobile close (when open) and the overlay is removed when resizing the browser to a larger size.

    Cheers 👍

    Marked as helpful
    0
    Priya Nayak•130
    @priya180975Posted over 2 years ago

    @bengera Thanks for the feedback

    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