Make modal isHidden as a function to return the current modal element state.

[marciotoshio]

This patch fix https://github.com/themesberg/flowbite/issues/119 by setting the isHidden state based on the current modal element state, without this we always assume that the model is hidden every time we create a new instance.

[marciotoshio]

Hmm this will not fix properly the issue, we set isHidden in a new instance and the old reference does not set the isHidden failing on subsequents calls because isHidden is all messed up now.

[marciotoshio]

Making the isHidden as a function we can always check the actual value of the element to decide its property instead of relying on the instance.

[zoltanszogyenyi]

Hey @marciotoshio,

Thanks for contributing, I'll check this out today!

[yaqshank92]

@zoltanszogyenyi Did you check this yet?

[zoltanszogyenyi]

Hey @yaqshank92,

Please check my latest comment here: https://github.com/themesberg/flowbite/issues/119

We have the v1.6.0 branch with some new updates, can you please test that out? Thanks!

[zoltanszogyenyi]

We have the isHidden() method on the Modal.

https://flowbite.com/docs/components/modal/#methods