Fixed error on container height computing when surroundingGutter is s…

…et to false. Removed bottom gutter to be consistent with top behaviour. Gutters are applied on the sides but not on top / bottom.
Spope · Aug 13, 2020 · 586a20e · 586a20e
1 parent 702ab22
commit 586a20e
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/build/minimasonry.esm.js b/build/minimasonry.esm.js
@@ -136,7 +136,7 @@ MiniMasonry.prototype.layout =  function() {
         this._columns[nextColumn]  += this._sizes[index] + (this._count > 1 ? this.conf.gutterY : this.conf.ultimateGutter);//margin-bottom
     }
 
-    this._container.style.height = (this._columns[this.getLongest()] - (this.conf.surroundingGutter ? 0 : this._conf.gutterY)) + 'px';
+    this._container.style.height = (this._columns[this.getLongest()] - this.conf.gutterY) + 'px';
 };
 
 MiniMasonry.prototype.getNextColumn = function(index) {

diff --git a/build/minimasonry.min.js b/build/minimasonry.min.js
diff --git a/src/minimasonry.js b/src/minimasonry.js
@@ -136,7 +136,7 @@ MiniMasonry.prototype.layout =  function() {
         this._columns[nextColumn]  += this._sizes[index] + (this._count > 1 ? this.conf.gutterY : this.conf.ultimateGutter);//margin-bottom
     }
 
-    this._container.style.height = (this._columns[this.getLongest()] - (this.conf.surroundingGutter ? 0 : this._conf.gutterY)) + 'px';
+    this._container.style.height = (this._columns[this.getLongest()] - this.conf.gutterY) + 'px';
 };
 
 MiniMasonry.prototype.getNextColumn = function(index) {