News Homepage with React, SCSS and JSON

Community feedback

• 

  wannacode•720

  @kwngptrlPosted over 1 year ago

  This is well done, but I have a few additional suggestions.

  In mobile view, if the menu is opened, perhaps you could position the X almost at the same position as the hamburger underneath. And since the content is still scrollable, you could either put the overlay over everything else and not just the height of the mobile menu. Or, you could disable scrolling.

  Marked as helpful

  0

  ---

  

  Martin Richter•160

  @Karag2006Posted over 1 year ago

  @kwngptrl You are absolutely right, i totally missed that you can scroll bejond the overlay. Will keep that in mind and maybe fix it later.

  Thanks.

  0