Add ability to pass Element itself to targetModal, not only 'id' of element about micromodal

[ghosh]

In an effort to keep this library lightweight and simple, have decided not to implement this feature. If there is a rising demand for this, am open to consider this again.

Thanks for your patience and understanding guys.

[pbek]

Being able to pass an element as modal would also make this great library work inside of web-components.

Otherwise you'll just get a

Uncaught TypeError: Cannot read property 'setAttribute' of null
    at Modal.showModal (micromodal.es.js:138)

I guess a simple this.modal = typeof targetModal === "string" ? document.getElementById(targetModal) : targetModal would already solve this issue.

[Raidri]

I push this because I need that feature too for my web components. Is there maybe any new informations?

[crtl]

Is this still on the radar?

[pbek]

@ghosh, I can confirm that changing this one line (https://github.com/ghosh/Micromodal/pull/351/files) allows the library to be used in web components.

[gstvribs]

any updates on this issue?

[obbap1]

please any updates on this ?

[ronilaukkarinen]

Bumping this as well. Not a big change.

[benCoomes]

Another bump, if this library doesn't support shadow dom then I can't use it for the project I am working on. Which is a shame, because it is forward-looking with no dependencies and es6 module support.

[pbek]

Just as a reminder, I have my PR #351 running in production since July 2020 without any issues in our web components.

[pbek]

Any news or considerations on this, @ghosh?