app menu section does not scroll on mobile

[samagragupta]

When the app menu and team members section is below Calendar. The window does not scroll over the app menu and team members section.

[boss-contributions-bot]

Thanks @samagragupta, for raising the issue! 🙌

One of our mentors will revert on this soon. ✅

Star ⭐ this project and tweet 🐦 about BOSS 2020.

[thenamankumar]

Add loom recording or gif.

[samagragupta]

@hereisnaman

[jugshaurya]

@hereisnaman, would like to work if @samagragupta not working.

[samagragupta]

@jugshaurya I am working on that.

[thenamankumar]

@samagragupta how are you trying to solve this issue?

[jugshaurya]

@samagragupta @hereisnaman,I think simply adding a height of 100vh to scroll menu can solve this. I tried and it works

[thenamankumar]

@samagragupta @hereisnaman,I think simply adding a height of 100vh to scroll menu can solve this. I tried and it works

How will 100vh work for desktop? Please share a gif.

[jugshaurya]

@samagragupta @hereisnaman, I think simply adding a height of 100vh to scroll menu can solve this. I tried and it works

How will 100vh work for desktop? Please share a gif.

@hereisnaman, sorry my bad: max-height: 100vh can solve this. here is how:

[thenamankumar]

@samagragupta @hereisnaman, I think simply adding a height of 100vh to scroll menu can solve this. I tried and it works

How will 100vh work for desktop? Please share a gif.

@hereisnaman, sorry my bad: max-height: 100vh can solve this. here is how:

Go ahead, create a PR.

[samagragupta]

@hereisnaman @jugshaurya From that we can scroll the team members section only. We cannot scroll the window.
I think we can also wheelPropagation true so that it can also scroll the window.
Should I make a PR @hereisnaman
Like this:

[jugshaurya]

@samagragupta no I dont think scroll window is going to be a problem. try adding the main section again (just for testing). we can scroll the window. I don't think there is a problem with that.

[thenamankumar]

@samagragupta @jugshaurya Lets discuss both the implementations in detail. You both raise the PR.

I think enabling wheel propagation will be better. I'll compare them locally once both the PRs are raised.

[jugshaurya]

@hereisnaman please review

[samagragupta]

@samagragupta @jugshaurya Lets discuss both the implementations in detail. You both raise the PR.

I think enabling wheel propagation will be better. I'll compare them locally once both the PRs are raised.

Okay, I will create the PR with my implementation.