Eui: EuiCollapsibleNav isDocked state does not work with manual toggle isOpen

[TAYTS]

I was trying to achieve manual toggle using isOpen with isDocked as the default behaviour for open state <EuiCollapsibleNav isOpen{visible} isDocked={visible} hideButtonIfDocked={false} > but if I toggle the Nav to close state and reduce the window size small enough (<992) and try to open the Nav again, the overlay is not display and unable to close the Nav. Just roughly read the code, it seems the windowIsLargeEnoughToDock might not set properly due to the removal to window listener when the isDocked is set to false earlier

[cchaos]

Do you have a codesandbox or an example gist we can take a look at for this particular configuration?

[TAYTS]

@cchaos Here is the codesandbox link: EuiCollapsibleNav_ManualToggleDemo

Steps to replicate the issue:

1. Click the toggle button to close the EuiCollapsibleNav
2. Reduce the window size (> 992px)
3. Click the toggle button again
4. Unable to close the EuiCollapsibleNav with the close empty button and no overlay mask shown

**Sorry there is some icon assets import issue.

[cchaos]

I'm understanding your configuration now. You are forcing the collapsible nav flyout to always push the body contents (always docked) when toggling it to visible. In other words, your nav will always be docked unless its forced to mobile overlay. I have found the problem area and will have a PR soon. I will also look into allowing the customization of that breakpoint.

[TAYTS]

@cchaos Thank you so much, I could try to help out for the breakpoint feature if you need

[cchaos]

@TAYTS Pr is here: #3330 If you want to take a look

[TAYTS]

@cchaos That's really fast, I can't thank you enough for the updates

[cchaos]

You're very welcome. If you find any more bugs just let us know, or if you feel comfortable, pass us a PR!