Frozen Rows not scrolling up and down properly

[6pac]

Example example-frozen-rows.html exhibits strange scrolling behaviour when selecting froxen rows at the top. See GIF.

I've been trying to track down the cause of this, but thought you might know it immediately @ghiscoding ?

[ghiscoding]

try rolling back my PR from yesterday and see if that makes any difference, it might be the cause... actually, probably not since it's only enabled when it's not frozen row. It seems like the scrollbar height is not calculated properly when using top frozen rows. I don't really know why, it might be another new bug we introduced after removing jQueryUI

[ghiscoding]

moving the comment I wrote in #761

While we're on a roll, #737 still appears to be a problem. Happy to look, but you just seem to be 'in the zone' right now, so I suspect what might take me several hours you may be able to do in 15 minutes.

I think this one is cause by the bottom container not being entirely filled except when you start scrolling then it adds more slick rows, a potential fix would be trigger a scroll event at same position when frozen row is changed to bottom.

Actually taking a quick look at it, I see that the problem is because the height is not recalculated properly after changing the frozen row dynamically. If you use this css style height: 1.24988e+06px; width: 1040px; (which is a copy of the grid-canvas grid-canvas-top grid-canvas-left and you apply it to the bottom (after changing frozen row that is), it will work as expected

from this

to this

the problem is I don't know how that height is being calculated correctly when we initialize the grid but not after. A potential quick hack fix would be to copy the one from the top into the bottom and vice versa when you do the inverse but that has a potential issue when user also changes the number of frozen rows (not just top/bottom toggle).... so anyway, I have identified the issue but not a real fix 😝

I actually wonder if that height that I wrote about is actually these ones from the default options

SlickGrid/slick.grid.js

Lines 115 to 116 in 21fe941

	ffMaxSupportedCssHeight: 6000000,
	maxSupportedCssHeight: 1000000000,

[ghiscoding]

closed by #774