[Question] Is it correct that overlay.current exists even if I unmount, unmountAll? about overlay-kit HOT 3 CLOSED

You’re correct in your observation.

To explain the background of this implementation: initially, during the PoC (Proof of Concept) phase of this library, the add and open steps of the reducer were combined into a single step, and similarly, the close and unmount steps were combined into one.

As a result, calling closeAll would trigger unmountAll immediately afterwards. In very specific scenarios, we noticed an issue where the animation would break. To temporarily fix this, we found that setting the current value helped resolve the issue.

To properly address this animation issue, we later decided to separate the add and open steps, as well as the close and unmount steps.

In other words, now that the close and unmount steps are separated, there’s no longer a need for the current value to point to the last modal. We will update this to point to null instead.

from overlay-kit.

The part you mentioned about overlayId potentially being an empty string was not intentional. It could lead to bugs.

The possibility of overlayId being an empty string could arise when a custom ID is specified. We need to prevent setting an empty string as a custom ID.

from overlay-kit.

Oh, Thank you for your detailed explanation!
I now have a better understanding of the library. 🙇‍♂️

from overlay-kit.