Skip to content

Commit

Permalink
Removed babel polyfill
Browse files Browse the repository at this point in the history
no issue
- the polyfill is no longer required for latest browsers (tested on Chrome, FF, and Safari)
- Edge may have problems but it's not currently supported
- reduces build size. Before/after:
  - `vendor.min.js: 3.29 MB (706 KB gzipped)`
  - `vendor.min.js: 3.2 MB (672.92 KB gzipped)`
  • Loading branch information
kevinansfield committed Jan 22, 2019
1 parent 509b16f commit 6efe2c8
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion ember-cli-build.js
Original file line number Diff line number Diff line change
Expand Up @@ -102,7 +102,7 @@ module.exports = function (defaults) {
let app = new EmberApp(defaults, {
'ember-cli-babel': {
optional: ['es6.spec.symbols'],
includePolyfill: true
includePolyfill: false
},
'ember-composable-helpers': {
only: ['toggle']
Expand Down
4 changes: 2 additions & 2 deletions lib/koenig-editor/addon/components/koenig-editor.js
Original file line number Diff line number Diff line change
Expand Up @@ -221,8 +221,8 @@ export default Component.extend({
// add our default atoms and cards, we want the defaults to be first so
// that they can be overridden by any passed-in atoms or cards.
// Use Array.concat to avoid modifying any passed in array references
atoms = Array.concat(defaultAtoms, atoms);
cards = Array.concat(defaultCards, cards);
atoms = defaultAtoms.concat(atoms);
cards = defaultCards.concat(cards);

return assign({
placeholder: this.placeholder,
Expand Down

0 comments on commit 6efe2c8

Please sign in to comment.