search

ckeditor / ckeditor4-angular

Official CKEditor 4 Angular component.
Other
55 stars 32 forks source link

Enable IE polyfills #5

Closed jacekbogdanski closed 5 years ago

jacekbogdanski commented 5 years ago

Using integration on IE9-11 requires enabling polyfills at src/polyfills.ts. We need to do some research to make sure that it's working correctly and if possible do it conditionally depending on the used browser. Angular may already cover this case - confirmation needed.

jacekbogdanski commented 5 years ago

Importing polyfills may be also totally on the user side.

engineering-this commented 5 years ago

In my opinion it should be only on the user side. We provide editor with nice support for older IEs. Within integration we use only API provided by Angular, so our component is limited to what the frameworks offer.

f1ames commented 5 years ago

I agree with @engineering-this, that polyfills should be on user/developer side. So if the app should support older IEs, then one can include polyfills. It's not the integration to make such decisions.