Horizontal header scrolling broken with `autoHeight=true`

When the option autoHeight=true is set headers do not scroll horizontally when the grid content is scrolled via the scrollbars.

This can be seen in example 11 - No vertical scrolling (autoHeight). Here’s a GIF showing the behaviour (note I edited the CSS to shorten the height):

Scrolling via the mousewheel does work (because that doesn’t call handleScroll()). In the GIF you can see the headers snap to the correct position when scrolling vertically.

This appears to be caused by the fix to #674, which assumes no scroll handling is required when autoHeight=true.

The fix for #674 should be reverted and the following change made in _handleScroll() instead:

- if (vScrollDist) {
+ if (vScrollDist && !options.autoHeight) {

This solves this issue and #674 remains fixed as well.

[6pac]

eeep. I clearly didn’t think of that use case when I made the patch! Thanks @timclutton will apply your fix as suggested. @ghiscoding I belive this will fix the original issue correctly.

[6pac]

@timclutton There is a new repo https://github.com/6pac/Slickgrid-2.x that will be maintained for a while for those who aren’t ready to do the whole update of the background files. I’ll apply ongoing non-3.0 related fixes there for as long as it’s not too difficult.