React-select: menuShouldBlockScroll also blocks menu scrolling.

[2Steaks]

Hi, I'm using menuPortalTarget to render outside of containers with overflow hidden such as table rows. In order for the menu position to be correct I have to set menuPosition to fixed. This works great except when you scroll the page and the menu position is not recalculated. To fix this I then use menuShouldBlockScroll which in combination with the other props achieves the desired result except for the fact that menuShouldBlockScroll also blocks the menu (has max-height with scroll auto) contents from being scrollable.

Would it be possible to prevent scrolling for all elements except for .react-select__menu-list ?

[flexdinesh]

Hey @2Steaks would you be replicate the issue in a codesandbox? It'd be easier to comprehend the issue with an example.

[2Steaks]

Good shout @flexdinesh, here you go: https://codesandbox.io/s/festive-dirac-izf5y?fontsize=14&hidenavigation=1&theme=dark

[epotockiy]

Confirmed the same issue for me, @flexdinesh do you check this bug in sandbox?
I made investigation and figured out, that this were broken after release of 3.0.5 version
I checked commits, and find this commit
When i reverted this changes locally, scroll start working again
I don't know reason of this changes, but i think that this should be reverted or updated

[stnwk]

Hi @JedWatson can you give any more insights into this? This is a big bummer :(

[flexdinesh]

Hey @2Steaks thanks for the codesandbox I can see it now. I'll get to it sometime soon. I'll update here once we make progress.

[srinivasmerugu]

Hey @2Steaks thanks for the codesandbox I can see it now. I'll get to it sometime soon. I'll update here once we make progress.

Hi @2Steaks Please let us know once it is pushed.

[engPabloMartinez]

Hey team, any news around a fix for this? maybe a temporary one? is going back to 3.0.4 the only workaround for now?

[2Steaks]

@flexdinesh reverting this commit seems to fix the issue.

https://github.com/JedWatson/react-select/commit/691a01171969da89e20097f9d603651bb18a8510#diff-16745c8f19a824159168a93abf7bb78b

[srinivasmerugu]

@flexdinesh Even we are waiting for this fix as we are using 3.1.0 features as well and do not want to go back to 3.04 version. Please let us know once you revert the 691a011#diff-16745c8f19a824159168a93abf7bb78b or give any other solution .

Thank you !

[bladey]

Hello -

Thanks for reporting this issue.

This bug has been reported multiple times as per issue #4088.

A new master issue #4088 has since been created to keep track this this bug.

This new issue will exist as the source of truth going forward to investigate the issue, report findings, and implement a bug fix.

We'll take into account all the details above while investigating.

If you feel this issue has been wrongly closed as it isn't related to the new master issue #4088, please let us know and we'll take another look.

Thank you!