remove(); not supported by IE about ngx-drag-drop

[brayunm]

Thanks for the quick response.

I've implemented the polyfill you've suggested and its working as expected.

[reppners]

Thanks for reporting! Didn't know about missing support of remove() in IE.

I'm leaning to point people towards applying a polyfill when IE support is needed:
https://developer.mozilla.org/en-US/docs/Web/API/ChildNode/remove

// from:https://github.com/jserz/js_piece/blob/master/DOM/ChildNode/remove()/remove().md
(function (arr) {
  arr.forEach(function (item) {
    if (item.hasOwnProperty('remove')) {
      return;
    }
    Object.defineProperty(item, 'remove', {
      configurable: true,
      enumerable: true,
      writable: true,
      value: function remove() {
        if (this.parentNode !== null)
          this.parentNode.removeChild(this);
      }
    });
  });
})([Element.prototype, CharacterData.prototype, DocumentType.prototype]);

The reason being I want to keep the code clean and not include workarounds needed for a single browser that in the future will become obsolete because it's not evergreen.

On the other hand using remove() is a code smell because its directly operating on the DOM. I'm thinking about refactoring the placeholder to <ng-template> if possible.

[reppners]

Will close this one because workaround/polyfill is available and there are issues that prevent using ng-template when nesting drop zones.