Dynamic scrollHeight for turboTable

[mseaman]

I noticed that if you change the scrollHeight dynamically in the turboTable which causes the scrollbar to show, it pushes the data to the left and then doesn't align with the header. This isn't the case on initiation or if it has a fixed scrollHeight.

I was able to fix it by calling this.alignScrollBar() in the source:

Code: Select all

    Object.defineProperty(ScrollableView.prototype, "scrollHeight", {
        get: function () {
            return this._scrollHeight;
        },
        set: function (val) {
            this._scrollHeight = val;
            this.setScrollHeight();
            this.alignScrollBar();
        },
        enumerable: true,
        configurable: true
    });

Do you think you can add this line or something similar to this property so that I won't need a custom version of TurboTable?

[cagatay.civici]

Provide a PR please and we'll review.