Removing Modernizr from build

It isn't necessary for such a simple site that is aimed at developers. I don't need to support outdated browsers that don't support @supports
Dan503 · Jan 8, 2018 · 9debbb3 · 9debbb3
1 parent ac3b85e
commit 9debbb3
Show file tree

Hide file tree

Showing 6 changed files with 2 additions and 42 deletions.
diff --git a/gulp/tasks/modernizr.js b/gulp/tasks/modernizr.js
diff --git a/gulpfile.babel.js b/gulpfile.babel.js
@@ -49,7 +49,6 @@ gulp.task('compile', gulp.series(
     'imagemin',
     'sass',
     'compile-tests',
-    'modernizr',
     'browserify',
   )
 ));

diff --git a/package.json b/package.json
@@ -54,7 +54,6 @@
     "gulp-imagemin": "^4.0.0",
     "gulp-jade": "^1.1.0",
     "gulp-load-plugins": "^1.5.0",
-    "gulp-modernizr": "0.0.0",
     "gulp-newer": "^1.3.0",
     "gulp-notify": "^3.0.0",
     "gulp-plumber": "^1.1.0",

diff --git a/src/_modules/tabs/tabs.scss b/src/_modules/tabs/tabs.scss
@@ -11,8 +11,4 @@
 			display: none;
 		}
 	}
-
-	.no-flexbox & {
-		width: 780px;
-	}
 }
diff --git a/src/_scripts/main.js b/src/_scripts/main.js
@@ -9,6 +9,8 @@ import codeLineNumbers from '~on-page-load-js/codeLineNumbers';
 import smoothAnchors from '~on-page-load-js/smoothAnchors';
 
 $(() => {
+  $('html').removeClass('no-js').addClass('js');
   codeLineNumbers();
   smoothAnchors();
+  $('html').addClass('-jsLoaded');
 });
diff --git a/src/_styles/04-base-styles/elements.scss b/src/_styles/04-base-styles/elements.scss
@@ -124,10 +124,6 @@ a {
     cursor: default;
     pointer-events: none;
     opacity: 0.2;
-
-    .no-csspointerevents & {
-      display: none;
-    }
   }
 }
 
@@ -199,10 +195,6 @@ pre {
     padding: 10px 5px;
     font-size: 1em;
   }
-
-  .no-flexbox & {
-    max-width: 780px;
-  }
 }
 
 code {