Merge pull request #1541 from roots/remove-modernizr

Remove Modernizr

assets/manifest.json

@@ -19,9 +19,6 @@
     },
     "jquery.js": {
       "bower": ["jquery"]
-    },
-    "modernizr.js": {
-      "bower": ["modernizr"]
     }
   },
   "config": {

bower.json

@@ -7,13 +7,9 @@
   "license": "MIT",
   "private": true,
   "dependencies": {
-    "modernizr": "2.8.2",
     "bootstrap-sass": "3.3.5"
   },
   "overrides": {
-    "modernizr": {
-      "main": "./modernizr.js"
-    },
     "bootstrap-sass": {
       "main": [
         "./assets/stylesheets/_bootstrap.scss",

lib/assets.php

@@ -4,13 +4,6 @@ namespace Roots\Sage\Assets;
 
 /**
  * Scripts and stylesheets
- *
- * Enqueue stylesheets in the following order:
- * 1. /theme/dist/styles/main.css
- *
- * Enqueue scripts in the following order:
- * 1. /theme/dist/scripts/modernizr.js
- * 2. /theme/dist/scripts/main.js
  */
 
 class JsonManifest {
@@ -72,7 +65,6 @@ function assets() {
     wp_enqueue_script('comment-reply');
   }
 
-  wp_enqueue_script('modernizr', asset_path('scripts/modernizr.js'), [], null, true);
   wp_enqueue_script('sage/js', asset_path('scripts/main.js'), ['jquery'], null, true);
 }
 add_action('wp_enqueue_scripts', __NAMESPACE__ . '\\assets', 100);