scrollIntoView doesn't work properly for some cases

@adamkleingit

Hi there,

I found one issue with the API call - scrollIntoView. For my case, it works most time with a small tree, but does not work for the large tree. So I created my own local function based on the source code below.

TreeNode.prototype.scrollIntoView = function () {
        if (this.elementRef) {
            var nativeElement = this.elementRef.nativeElement;
            nativeElement.scrollIntoViewIfNeeded && nativeElement.scrollIntoViewIfNeeded();
            return this;
        }
    };

I am using scrollIntoView instead of scrollIntoViewIfNeeded, which is also suggested by https://developer.mozilla.org/en-US/docs/Web/API/Element/scrollIntoViewIfNeeded

My code:

        if (this.focusedNode) {
          if (this.focusedNode.elementRef) {
            let nativeElement = this.focusedNode.elementRef.nativeElement;
            nativeElement.scrollIntoView();
          }
        };

It works for both small and large trees.

I hope my finding and solution will be helpful to others.

Best

-John

[ottosson]

I’ve got the same issue. scrollIntoView works in Firefox and IE11 for my application, but not in Chrome. I can’t seem to reproduce a working example to show here though.

Here is a broken example where scrollIntoView doesn’t work in any browser I’ve tried with.

[phoude]

It did the trick. Thanks!