syncfusion/ej2-javascript-ui-controls

this.lastPos scope is window and not TreeInterSectionObserver

csnyder8749 opened this issue · 3 comments

csnyder8749 commented

The clear method is being called out of scope.

This is error occurring everytime we scroll in a TreeGrid with virtualization:
Uncaught TypeError: Cannot set properties of undefined (setting 'lastPos')

This seems to be the issue
export class TreeInterSectionObserver extends InterSectionObserver , but has no constructor and does not have the following

    constructor() {
        super();
    }

In TypeScript when a class extends another class, the constructor of the base class "super' needs to be called or the inheriting class will not be created correctly. This is being done correctly in other classes in this file so it looks like an error of omission.

this is the constructor of InterSectionObserver

    constructor(element: HTMLElement, options: InterSection, movableEle?: HTMLElement) {
        this.element = element;
        this.options = options;
        this.movableEle = movableEle;
    }

intersection-observer.ts

farveensulthana commented

Hi Craig Snyder,

Greetings from Syncfusion support.

Before proceeding your query, we need some additional information to replicate the issue from our end. Kindly share the following details to proceed further.

  1. Complete treegrid code details (properties and events, columns definition and events handling details if any).
  2. Video demo/screenshot of the issue.
  3. Share the platform that you are using with version.
  4. Share the package.json file to know about Syncfusion package version details.
  5. Exact replication steps to replicate the issue from our end(using video Demo.
  6. Are you using virtual with frozen columns or faced issue on enabling specific properties?
  7. If possible, kindly replicate the issue from this sample.

Kindly get back to us with the above details to proceed further.

Regards,

Farveen sulthana T

gsumankumar commented

This issue has been inactive for over a week, so we are closing it for housekeeping purposes. If you have any additional information to assist us, please reopen the issue. Thank you for your understanding.