From be996b200977642ed0586fb899f0cd31bdf8c4d5 Mon Sep 17 00:00:00 2001
From: ghiscoding <gbeaulac@gmail.com>
Date: Sat, 13 May 2023 13:54:33 -0400
Subject: [PATCH] fix: toggling frozen rows should recalc scroll height, closes
 #737 - when changing frozen rows via `setOptions`, it should recalculate each
 viewports (top/bottom) - the previous code skipped scroll height
 recalculation and that caused the issue identified in #737

---
 slick.grid.js | 9 ++++++++-
 1 file changed, 8 insertions(+), 1 deletion(-)

diff --git a/slick.grid.js b/slick.grid.js
index db8d2a8a..79655803 100644
--- a/slick.grid.js
+++ b/slick.grid.js
@@ -306,6 +306,7 @@ if (typeof Slick === "undefined") {
     var cssShow = { position: 'absolute', visibility: 'hidden', display: 'block' };
     var _hiddenParents;
     var oldProps = [];
+    var enforceFrozenRowHeightRecalc = false;
     var columnResizeDragging = false;
     var slickDraggableInstance = null;
     var slickMouseWheelInstances = [];
@@ -3221,6 +3222,11 @@ if (typeof Slick === "undefined") {
       validateAndEnforceOptions();
       setFrozenOptions();
 
+      // when user changed frozen row option, we need to force a recalculation of each viewport heights
+      if (args.frozenBottom !== undefined) {
+        enforceFrozenRowHeightRecalc = true;
+      }
+
       _viewport.forEach(function (view) {
         view.style["overflow-y"] = options.autoHeight ? "hidden" : "auto";
       });
@@ -3970,7 +3976,7 @@ if (typeof Slick === "undefined") {
         }
       }
 
-      if (h !== oldH) {
+      if (h !== oldH || enforceFrozenRowHeightRecalc) {
         if (hasFrozenRows && !options.frozenBottom) {
           utils.height(_canvasBottomL, h);
 
@@ -3983,6 +3989,7 @@ if (typeof Slick === "undefined") {
         }
 
         scrollTop = _viewportScrollContainerY.scrollTop;
+        enforceFrozenRowHeightRecalc = false; // reset enforce flag
       }
 
       var oldScrollTopInRange = (scrollTop + offset <= th - tempViewportH);