Responsive News Homepage

Community feedback

• 

  mrayala42(Angel)•120

  @coderboi559Posted almost 2 years ago

  Hello, with the overlay and mobile menu. You can use position: fixed to cover everything up. Using media queries if you want it just for mobile. Overall pretty good job. I would use display: grid; to line everything up. Make 3 grid-columns and align everything according to the design to make it look cleaner.

  Marked as helpful

  1
• 

  Amal Karim ✅•1,290

  @amalkarimPosted almost 2 years ago

  Hi Amya,

  That's an interesting solution.

  If you want to make the overlay extends the whole height of the page, there's approach other than using javascript. I don't know if this is better solution or not, but I think it's more concise. Just add this to the existing style:

  body {
      ...
      position: relative;
  }
  .menu-overlay {
      ....
      height: 100%;
  }
  

  Change height of navigation mobile too:

  nav.navigation.mobile {
      ...
      height: 100%;
  }
  

  That's it. Hope this helps. Happy coding!

  Marked as helpful

  1