fix: ensure modal={false} applies when using controlled open state

[y-solb]

What

Fixes an issue where modal={false} had no effect when using a controlled open prop.

Why

Internally, the modal check only applied on initial mount. When controlling the drawer state using the open prop, pointer events were not correctly restored on open.

How

Added isOpen as a dependency to the useEffect that applies pointer-events: auto when modal is false.

Related

No open issue yet, but related to modal mode accessibility and pointer control.

---

Thanks for this great library! 😊

[NicHaley]

Having issues with this too. Anything stopping this from getting merged?

[y-solb]

I ran into the same issue and had temporarily worked around it like this:

useEffect(() => {
  requestAnimationFrame(() => {
    document.body.style.pointerEvents = isOpen ? "auto" : "";
  });
}, [isOpen]);

Would love to see this merged! @NicHaley

[fpw23]

Thank you @y-solb this was my exact issue and your suggestion worked perfectly! If anyone else needs this I found the easiest way to fix it is with patch-package. Here is a good post about how to use it.

[bernatfortet]

Same here!

[ianlet]

Thank you @y-solb this was my exact issue and your suggestion worked perfectly! If anyone else needs this I found the easiest way to fix it is with patch-package. Here is a good post about how to use it.

For those using pnpm, there's directly a patch command in the CLI.