Merge pull request #505 from christophercr/feature/obsolete-deps

chore(stark-all): remove obsolete and unnecessary NPM dependencies
NationalBankBelgium · Jul 13, 2018 · 34af0ae · 34af0ae
2 parents 765e5ef + 3a6392d
commit 34af0ae
Show file tree

Hide file tree

Showing 5 changed files with 10 additions and 11 deletions.
diff --git a/packages/stark-build/package.json b/packages/stark-build/package.json
@@ -21,7 +21,6 @@
   "dependencies": {
     "@angular-devkit/build-angular": "0.6.8",
     "@angular-devkit/build-optimizer": "0.6.8",
-    "@types/source-map": "0.5.7",
     "@types/uglify-js": "3.0.2",
     "@types/webpack": "4.4.5",
     "add-asset-html-webpack-plugin": "2.1.3",

diff --git a/showcase/package.json b/showcase/package.json
@@ -137,7 +137,6 @@
     "reflect-metadata": "0.1.12",
     "rxjs": "6.2.1",
     "rxjs-compat": "6.2.1",
-    "web-animations-js": "2.3.1",
     "zone.js": "0.8.26"
   },
   "devDependencies": {

diff --git a/showcase/src/polyfills.browser.ts b/showcase/src/polyfills.browser.ts
@@ -49,13 +49,14 @@ import "event-source-polyfill";
 import "eligrey-classlist-js-polyfill";
 
 /**
- * All but Chrome, Firefox and Opera require this to support Web Animations from
- * `@angular/animations` and `@angular/platform-browser/animations`.
- * See: http://caniuse.com/#feat=web-animation
+ * Web Animations polyfill is no longer needed for standard animation support as of Angular 6
+ * IMPORTANT: It is only needed in case you use the AnimationBuilder from '@angular/animations' in the application
  *
+ * See: https://angular.io/guide/browser-support#optional-browser-features-to-polyfill
+ * See: http://caniuse.com/#feat=web-animation
  * Polyfill: https://github.com/web-animations/web-animations-js
  */
-import "web-animations-js";
+// import "web-animations-js";
 
 /***************************************************************************************************
  * Zone JS is required by Angular itself.

diff --git a/starter/package.json b/starter/package.json
@@ -136,7 +136,6 @@
     "reflect-metadata": "0.1.12",
     "rxjs": "6.2.1",
     "rxjs-compat": "6.2.1",
-    "web-animations-js": "2.3.1",
     "zone.js": "0.8.26"
   },
   "devDependencies": {

diff --git a/starter/src/polyfills.browser.ts b/starter/src/polyfills.browser.ts
@@ -49,13 +49,14 @@ import "event-source-polyfill";
 import "eligrey-classlist-js-polyfill";
 
 /**
- * All but Chrome, Firefox and Opera require this to support Web Animations from
- * `@angular/animations` and `@angular/platform-browser/animations`.
- * See: http://caniuse.com/#feat=web-animation
+ * Web Animations polyfill is no longer needed for standard animation support as of Angular 6
+ * IMPORTANT: It is only needed in case you use the AnimationBuilder from '@angular/animations' in the application
  *
+ * See: https://angular.io/guide/browser-support#optional-browser-features-to-polyfill
+ * See: http://caniuse.com/#feat=web-animation
  * Polyfill: https://github.com/web-animations/web-animations-js
  */
-import "web-animations-js";
+// import "web-animations-js";
 
 /***************************************************************************************************
  * Zone JS is required by Angular itself.