htmlbars icon indicating copy to clipboard operation
htmlbars copied to clipboard

Add temporary workaround for SimpleDOM

Open chancancode opened this issue 9 years ago • 2 comments

Requires https://github.com/krisselden/morph-range/pull/8

The current parseHTML implementation requires some DOM APIs not implemented in SimpleDOM. This commit adds a temporary workaround to support using SimpleDOM on the server.

In the future, we will move towards using insertAdjacentHTML and innerHTML= as the main APIs and remove the need of parseHTML altogether.

This commit also reverts a previous commit that introduced the setMorphHTML method, which does exactly what Morph#setHTML does.

(I added a test, but it's not currently running because we disabled all the node tests since https://github.com/tildeio/htmlbars/pull/359... @stefanpenner ? Anyhow, this particular change is very easy to verify by :eyes:, so it's not critical to re-enable them just to merge this PR.)

chancancode avatar Oct 06 '15 08:10 chancancode

@krisselden @wycats can you take a look? :eyes:

chancancode avatar Oct 06 '15 08:10 chancancode

Hey, what's the status of this? :) @chancancode

e00dan avatar Aug 19 '16 22:08 e00dan