You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Per #49 / #50 , it seems that there was a much easier solution to the problem of wanting static content in the Light DOM all along; just use innerHTML 😅
Details
I think given how using innerHTML is just standard DOM and per #50 allows each component to opt-in individually, it seems to make this configuration option irrelevant, for now at least. So should
Refactor website to use innerHTML
Update compiler to drop support for lightMode configuration and remove from build.js
Update documentation to remove reference to lightMode
Open up a discussion to capture the original intent / use case, just for posterity in case its something worth re-investigating
Update README
Not sure if this implicates this getInnerHTML issue at all?
The text was updated successfully, but these errors were encountered:
Type of Change
Summary
Per #49 / #50 , it seems that there was a much easier solution to the problem of wanting static content in the Light DOM all along; just use
innerHTML
😅Details
I think given how using
innerHTML
is just standard DOM and per #50 allows each component to opt-in individually, it seems to make this configuration option irrelevant, for now at least. So shouldinnerHTML
lightMode
configuration and remove from build.jslightMode
getInnerHTML
issue at all?The text was updated successfully, but these errors were encountered: