Fixed Gulp build pipeline

CHANGELOG

@@ -1,3 +1,15 @@
+mkdocs-material-1.1.0 (2017-xx-xx)
+
+  * Added static type checking using Facebook's Flow
+  * Added Disqus integration (optional)
+  * Added tabs navigation feature (optional)
+  * Added a high resolution Favicon with the new logo
+  * Fixed #175: Tables cannot be set to 100% width
+  * Fixed #173: Dictionary elements have no bottom spacing
+  * Fixed race conditions in build related to asset revisioning
+  * Refactored and automated Docker build and PyPI release
+  * Refactored styles related to tables
+
 mkdocs-material-1.0.5 (2017-02-18)
 
   * Fixed #153: Sidebar flows out of constrained area in Chrome 56

Gulpfile.babel.js

@@ -154,28 +154,28 @@ gulp.task("assets:images:clean",
 /*
  * Build application logic
  *
- * When revisioning, the build must be serialized due to race conditions
- * happening when two tasks try to write manifest.json simultaneously
+ * When revisioning assets, the build must be serialized due to possible race
+ * conditions when two tasks try to write manifest.json simultaneously
  */
 
 gulp.task("assets:javascripts:build:application", [
-  args.revision ? "assets:javascripts:build:application" : false,
   args.clean ? "assets:javascripts:clean" : false,
-  args.lint ? "assets:javascripts:lint" : false
+  args.lint ? "assets:javascripts:lint" : false,
+  args.revision ? "assets:stylesheets:build" : false
 ].filter(t => t),
   load("assets/javascripts/build/application"))
 
 /*
  * Build custom modernizr
  *
- * When revisioning, the build must be serialized due to race conditions
- * happening when two tasks try to write manifest.json simultaneously
+ * When revisioning assets, the build must be serialized due to possible race
+ * conditions when two tasks try to write manifest.json simultaneously
  */
 gulp.task("assets:javascripts:build:modernizr", [
   "assets:stylesheets:build",
-  args.revision ? "assets:javascripts:build:application" : false,
   args.clean ? "assets:javascripts:clean" : false,
-  args.lint ? "assets:javascripts:lint" : false
+  args.lint ? "assets:javascripts:lint" : false,
+  args.revision ? "assets:javascripts:build:application" : false
 ].filter(t => t),
   load("assets/javascripts/build/modernizr"))
 
@@ -261,10 +261,10 @@ gulp.task("assets:clean", [
  */
 
 gulp.task("views:build", [
+  args.clean ? "views:clean" : false,
   args.revision ? "assets:images:build" : false,
   args.revision ? "assets:stylesheets:build" : false,
-  args.revision ? "assets:javascripts:build" : false,
-  args.clean ? "views:clean" : false
+  args.revision ? "assets:javascripts:build" : false
 ].filter(t => t),
   load("views/build"))
 
@@ -285,8 +285,7 @@ gulp.task("mkdocs:build", [
   "assets:build",
   "views:build",
   "mkdocs:clean"
-].filter(t => t),
-  load("mkdocs/build"))
+], load("mkdocs/build"))
 
 /*
  * Clean documentation build

material/base.html

@@ -32,7 +32,7 @@
       <script src="{{ base_url }}/assets/javascripts/modernizr-56ade86843.js"></script>
     {% endblock %}
     {% block styles %}
-      <link rel="stylesheet" href="{{ base_url }}/assets/stylesheets/application-a7dac97dbb.css">
+      <link rel="stylesheet" href="{{ base_url }}/assets/stylesheets/application-954a1bc95b.css">
       {% if config.extra.palette %}
         <link rel="stylesheet" href="{{ base_url }}/assets/stylesheets/application-02ce7adcc2.palette.css">
       {% endif %}
@@ -143,7 +143,7 @@
       {% endblock %}
     </div>
     {% block scripts %}
-      <script src="{{ base_url }}/assets/javascripts/application-8dc3dfc020.js"></script>
+      <script src="{{ base_url }}/assets/javascripts/application-6da1e0a63e.js"></script>
       <script>app.initialize({url:{base:"{{ base_url }}"}})</script>
       {% for path in extra_javascript %}
         <script src="{{ path }}"></script>

scripts/start

@@ -28,4 +28,5 @@ if [[ ! -d `npm bin` ]]; then
 fi
 
 # Run command
+`npm bin`/gulp clean && \
 `npm bin`/gulp watch --no-lint "$@"

src/assets/javascripts/components/Material/Header/Shadow.js

@@ -42,14 +42,16 @@ export default class Shadow {
     const ref = (typeof el === "string")
       ? document.querySelector(el)
       : el
-    if (!(ref instanceof Node) ||
-        !(ref.parentNode instanceof HTMLElement) ||
-        !(ref.parentNode.previousElementSibling instanceof HTMLElement))
+    if (!(ref instanceof HTMLElement) ||
+        !(ref.parentNode instanceof HTMLElement))
       throw new ReferenceError
 
     /* Grab parent and header */
     this.el_ = ref.parentNode
-    this.header_ = ref.parentNode.previousElementSibling
+    if (!(this.el_.parentNode instanceof HTMLElement) ||
+        !(this.el_.parentNode.previousElementSibling instanceof HTMLElement))
+      throw new ReferenceError
+    this.header_ = this.el_.parentNode.previousElementSibling
 
     /* Initialize height and state */
     this.height_ = 0

src/assets/stylesheets/layout/_sidebar.scss

@@ -112,6 +112,9 @@
     max-height: 100%;
     margin: 0 0.4rem;
     overflow-y: auto;
+    // Hack: putting the scroll wrapper on the GPU massively reduces jitter
+    // when locking the sidebars into place
+    backface-visibility: hidden;
 
     // [tablet -]: Adjust margins
     @include break-to-device(tablet) {