From d05e3370d05562225c76aad19ea9fe7f1bce0303 Mon Sep 17 00:00:00 2001 From: ghiscoding Date: Tue, 18 May 2021 15:37:19 -0400 Subject: [PATCH] fix: `setOptions` should call `setOverflow` when changing frozenColumns - if we change from a regular grid to a frozen grid dynamically by calling `setOptions({ frozenColumn: 2 })` then it should call `setOverflow` to redo the associated css overflow. This PR does just that and prior to this PR the overflow was just reset and sometime the scrolling was showing up on the left section when it shouldn't --- slick.grid.js | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/slick.grid.js b/slick.grid.js index 0ff01454..73c3bc91 100644 --- a/slick.grid.js +++ b/slick.grid.js @@ -2910,7 +2910,7 @@ if (typeof Slick === "undefined") { return options; } - function setOptions(args, suppressRender, suppressColumnSet) { + function setOptions(args, suppressRender, suppressColumnSet, suppressSetOverflow) { if (!getEditorLock().commitCurrentEdit()) { return; } @@ -2938,6 +2938,9 @@ if (typeof Slick === "undefined") { setFrozenOptions(); setScroller(); + if (!suppressSetOverflow) { + setOverflow(); + } zombieRowNodeFromLastMouseWheelEvent = null; if (!suppressColumnSet) {