From: Kevin Ball <kmball11@gmail.com>
Date: Sun, 28 May 2017 03:39:53 +0000 (-0700)
Subject: Clean up bugs noted by Brett
X-Git-Tag: v6.4.0-rc1~9^2~50
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=a29f791145e45a5bb59042cbd4188bb07bbed200;p=thirdparty%2Ffoundation%2Ffoundation-sites.git

Clean up bugs noted by Brett
---

diff --git a/scss/zf-grid/_cell.scss b/scss/zf-grid/_cell.scss
index cb7c2d7e4..4c7d5666c 100644
--- a/scss/zf-grid/_cell.scss
+++ b/scss/zf-grid/_cell.scss
@@ -101,7 +101,7 @@
   // TODO: figure out how we want to define this
 }
 
-@mixin zf-cell-nonresponsive(
+@mixin zf-cell-static(
   $size: full,
   $gutter-output: true,
   $gutters:$grid-margin-gutters,
diff --git a/scss/zf-grid/_classes.scss b/scss/zf-grid/_classes.scss
index 8505857f1..eeda9d1c6 100644
--- a/scss/zf-grid/_classes.scss
+++ b/scss/zf-grid/_classes.scss
@@ -48,7 +48,7 @@
     @for $i from 1 through $grid-columns {
       // Sizing (percentage)
       .#{$-zf-size}-#{$i} {
-        @include zf-cell-nonresponsive($i, false);
+        @include zf-cell-static($i, false, $gutter-type: padding);
       }
     }
   }
@@ -65,48 +65,48 @@
 
     // Base cell styles
     > .cell {
-      @include zf-cell-nonresponsive($vertical: $vertical);
+      @include zf-cell-static($vertical: $vertical);
     }
 
     // Auto width
     @include -zf-each-breakpoint(false) {
       @if(type-of($grid-margin-gutters) == 'map' and map-has-key($grid-margin-gutters, $-zf-size)) {
         > .cell {
-          @include zf-cell-nonresponsive($breakpoint: $-zf-size, $vertical: $vertical);
+          @include zf-cell-static($breakpoint: $-zf-size, $vertical: $vertical);
         }
         > .auto {
-          @include zf-cell-nonresponsive(auto, false, $breakpoint: $-zf-size, $vertical: $vertical);
+          @include zf-cell-static(auto, false, $breakpoint: $-zf-size, $vertical: $vertical);
         }
         > .shrink {
-          @include zf-cell-nonresponsive(shrink, false, $breakpoint: $-zf-size, $vertical: $vertical);
+          @include zf-cell-static(shrink, false, $breakpoint: $-zf-size, $vertical: $vertical);
         }
       }
       > .#{$-zf-size}-auto {
-        @include zf-cell-nonresponsive(auto, false, $breakpoint: $-zf-size, $vertical: $vertical);
+        @include zf-cell-static(auto, false, $breakpoint: $-zf-size, $vertical: $vertical);
       }
       > .#{$-zf-size}-shrink {
-        @include zf-cell-nonresponsive(shrink, false, $breakpoint: $-zf-size, $vertical: $vertical);
+        @include zf-cell-static(shrink, false, $breakpoint: $-zf-size, $vertical: $vertical);
       }
 
       @for $i from 1 through $grid-columns {
         // Sizing (percentage)
         > .#{$-zf-size}-#{$i} {
-          @include zf-cell-nonresponsive($i, false, $breakpoint: $-zf-size, $vertical: $vertical);
+          @include zf-cell-static($i, false, $breakpoint: $-zf-size, $vertical: $vertical);
         }
       }
       @if(type-of($grid-margin-gutters) == 'map' and map-has-key($grid-margin-gutters, $-zf-size)) {
         @each $bp in -zf-breakpoints-less-than($-zf-size) {
           @if(map-has-key($grid-margin-gutters, $bp)) {
             > .#{$bp}-auto {
-              @include zf-cell-nonresponsive(auto, false, $breakpoint: $bp, $vertical: $vertical);
+              @include zf-cell-static(auto, false, $breakpoint: $bp, $vertical: $vertical);
             }
             > .#{$bp}-shrink {
-              @include zf-cell-nonresponsive(shrink, false, $breakpoint: $bp, $vertical: $vertical);
+              @include zf-cell-static(shrink, false, $breakpoint: $bp, $vertical: $vertical);
             }
             @for $i from 1 through $grid-columns {
               // Sizing (percentage)
               > .#{$bp}-#{$i} {
-                @include zf-cell-nonresponsive($i, false, $breakpoint: $bp, $vertical: $vertical);
+                @include zf-cell-static($i, false, $breakpoint: $bp, $vertical: $vertical);
               }
             }
           }
@@ -116,7 +116,7 @@
       @for $i from 1 through $grid-columns {
         // Sizing (percentage)
         > .#{$-zf-size}-#{$i} {
-          @include zf-cell-nonresponsive($i, false, $breakpoint: $-zf-size, $vertical: $vertical);
+          @include zf-cell-static($i, false, $breakpoint: $-zf-size, $vertical: $vertical);
         }
       }
     }
@@ -183,19 +183,19 @@
     @include -zf-each-breakpoint() {
       @if(type-of($grid-margin-gutters) == 'map' and map-has-key($grid-margin-gutters, $-zf-size)) {
         > .cell {
-          @include zf-cell-nonresponsive($breakpoint: $-zf-size, $vertical: true);
+          @include zf-cell-static($breakpoint: $-zf-size, $vertical: true);
         }
         > .auto {
-          @include zf-cell-nonresponsive(auto, false, $breakpoint: $-zf-size, $vertical: true);
+          @include zf-cell-static(auto, false, $breakpoint: $-zf-size, $vertical: true);
         }
         > .shrink {
-          @include zf-cell-nonresponsive(shrink, false, $breakpoint: $-zf-size, $vertical: true);
+          @include zf-cell-static(shrink, false, $breakpoint: $-zf-size, $vertical: true);
         }
       }
       @for $i from 1 through $grid-columns {
         // Sizing (percentage)
         .#{$-zf-size}-#{$i} {
-          @include zf-cell-nonresponsive($i, false, $vertical: true);
+          @include zf-cell-static($i, false, $vertical: true);
         }
       }
     }
diff --git a/scss/zf-grid/_collapse.scss b/scss/zf-grid/_collapse.scss
index e5a3990c0..53a951a23 100644
--- a/scss/zf-grid/_collapse.scss
+++ b/scss/zf-grid/_collapse.scss
@@ -22,18 +22,20 @@
 
     @each $value in $gutter-position {
       margin-#{$value}: 0;
+    }
 
       > #{$selector} {
-        margin-#{$value}: 0;
+        @each $value in $gutter-position {
+          margin-#{$value}: 0;
+        }
       }
-    }
 
     // Output new widths to not include gutters
     @each $bp in $breakpoint-classes {
       @for $i from 1 through $grid-columns {
         // Sizing (percentage)
         > .#{$bp}-#{$i} {
-          @include zf-cell-nonresponsive($i, $gutter-output: false, $gutter-type: padding);
+          @include zf-cell-static($i, $gutter-output: false, $gutter-type: padding);
         }
       }
     }
@@ -42,19 +44,11 @@
 
     @each $value in $gutter-position {
       margin-#{$value}: 0;
-
-      > #{$selector} {
-        padding-#{$value}: 0;
-      }
     }
 
-    // Output new widths to not include gutters
-    @each $bp in $breakpoint-classes {
-      @for $i from 1 through $grid-columns {
-        // Sizing (percentage)
-        > .#{$bp}-#{$i} {
-          @include zf-cell-nonresponsive($i, $gutter-output: false, $gutter-type: padding);
-        }
+    > #{$selector} {
+      @each $value in $gutter-position {
+        padding-#{$value}: 0;
       }
     }
   }