Polyfill makes changes to the DOM beyond the image editor component

[cgratie]

Describe the bug

The svgxuse polyfill makes changes to all the <use> tags in the document. This can break other functionality when the image editor is integrated into a larger project. It is also not compatible with React, because the DOM changes are likely to interfere with the reconciliation algorithm.

In my specific case this also creates a bug: SVG sprites used outside of the image editor are not loading in certain cases.

To Reproduce

1.import ImageEditor from '@toast-ui/react-image-editor'; somewhere inside a CRA webapp.
2. Somewhere else, use an SVG sprite as <svg><use href="https://url/of/svg#hash" /><svg>.
3. Start the dev server.
4. In the browser, set network throttling to 3G and reload the page.
5. The SVG sprite will not load. In the inspector we can also see that the href attribute has been overwritten to hold only "#hash".

Expected behavior

• No change of the DOM that goes beyond the image editor component :)
• OR the option to disable this polyfill when using React

Desktop (please complete the following information):

• OS: [e.g. Ubuntu 20.04
• Browser Chrome
• Version 98

Additional context

In my project I have confirmed that the issue is directly related to this polyfill by editing the corresponding file to remove it.
I am also able to "disable" the polyfill with the following aggressive workaround:

window.MutationObserver = () => ({ observe: () => {} });

[lja1018]

@cgratie
Thank you for the report.
The polyfill is required for external URL loading in IE. Therefore, a different approach is needed rather than the deletion of the polyfill.
It seems that the examples presented may be improved and used.

[cgratie]

@lja1018
I see, thank you for the clarification.
So maybe a solution can be to apply the polyfill only for the case of IE (if it is not needed on other browsers)? Or, instead, to provide some config option for enabling/disabling it and let users of the package decide when to activate it.

[lja1018]

@cgratie
We will consider applying polyfill as an option.
Thank you.