SyncRowHeight - #7417

Discuss issues related to v5.x
User avatar
bee
Posts: 9
Joined: Mon Oct 09, 2017 3:29 pm

Re: SyncRowHeight - #7417

Post by bee » Wed Feb 26, 2020 10:55 am

Like already described in viewtopic.php?f=49&t=10274&sid=4f42eeac7d2be24c7b02c99f69a56eb6&start=10
the bug still exists using the latest version

I think its not necessary to reply in both posts, so i hope the issue will be fixed :)

User avatar
mats
Core Developer
Core Developer
Posts: 15590
Joined: Sat Dec 19, 2009 11:41 pm
Location: Sweden
Contact:

Re: SyncRowHeight - #7417

Post by mats » Thu Feb 27, 2020 10:42 am

Issue is now resolved in our 6.x branch, please try the next nightly build and let us know if all is ok now.
Tired of debugging javascript errors in web applications? Try our new error logging service RootCause, or read more on the Sencha blog

@bryntum
Facebook
API documentation

User avatar
bee
Posts: 9
Joined: Mon Oct 09, 2017 3:29 pm

Re: SyncRowHeight - #7417

Post by bee » Sun Mar 01, 2020 2:50 pm

Using the latest nightly build all of the resources and events are displayed but adding/removing events from the scheduler and scrolling afterwards now rarely leads to an error:

Code: Select all

Uncaught TypeError: Cannot read property 'getAttribute' of null
    at F.clearViewEl (scripts.3f284166dbef1ab0d786.js:1)
    at F.clearViewEl (scripts.3f284166dbef1ab0d786.js:1)
    at F.onRangeFetched (scripts.3f284166dbef1ab0d786.js:1)
    at F.t.onRangeFetched (scripts.3f284166dbef1ab0d786.js:1)
    at F.renderRange (scripts.3f284166dbef1ab0d786.js:1)
    at F.handleViewScroll (scripts.3f284166dbef1ab0d786.js:1)
    at F.onViewScroll (scripts.3f284166dbef1ab0d786.js:1)
    at F.fire (scripts.3f284166dbef1ab0d786.js:1)
    at F.doFireEvent (scripts.3f284166dbef1ab0d786.js:1)
    at F.fireEventArgs (scripts.3f284166dbef1ab0d786.js:1)
which breaks the whole scheduler

(the flag "syncRowHeight" is not used/enabled)

User avatar
mats
Core Developer
Core Developer
Posts: 15590
Joined: Sat Dec 19, 2009 11:41 pm
Location: Sweden
Contact:

Re: SyncRowHeight - #7417

Post by mats » Mon Mar 02, 2020 10:16 am

Test case? Please provide exact instructions and we'll investigate
Tired of debugging javascript errors in web applications? Try our new error logging service RootCause, or read more on the Sencha blog

@bryntum
Facebook
API documentation

User avatar
mats
Core Developer
Core Developer
Posts: 15590
Joined: Sat Dec 19, 2009 11:41 pm
Location: Sweden
Contact:

Re: SyncRowHeight - #7417

Post by mats » Mon Mar 02, 2020 1:00 pm

It works perfectly here, adding removing events etc. If you can upload something we can run we'll look at it asap
Tired of debugging javascript errors in web applications? Try our new error logging service RootCause, or read more on the Sencha blog

@bryntum
Facebook
API documentation

User avatar
bee
Posts: 9
Joined: Mon Oct 09, 2017 3:29 pm

Re: SyncRowHeight - #7417

Post by bee » Wed Mar 04, 2020 9:22 am

unfortunately i cant provide you with a test case because i dont know what causes the issue. With simple setups i cant reproduce the behaviour and the error rarely pops up.

In our scheduler events asynchrously (result of rest requests or based on push events via websocket) get removed or inserted (store.add/store.remove). We never had any issues.
Another error which rarely throws during scrolling:

Code: Select all

Uncaught TypeError: Cannot read property 'offsetTop' of undefined
    at F.getLastVisibleRowIndex (scripts.3f284166dbef1ab0d786.js:1)
    at F.getLastVisibleRowIndex (scripts.3f284166dbef1ab0d786.js:1)
    at F.handleViewScroll (scripts.3f284166dbef1ab0d786.js:1)
    at F.onViewScroll (scripts.3f284166dbef1ab0d786.js:1)
    at F.fire (scripts.3f284166dbef1ab0d786.js:1)
    at F.doFireEvent (scripts.3f284166dbef1ab0d786.js:1)
    at F.fireEventArgs (scripts.3f284166dbef1ab0d786.js:1)
    at F.fireEvent (scripts.3f284166dbef1ab0d786.js:1)
    at F.fireScroll (scripts.3f284166dbef1ab0d786.js:1)
    at F.updateDomScrollPosition (scripts.3f284166dbef1ab0d786.js:1)

User avatar
mats
Core Developer
Core Developer
Posts: 15590
Joined: Sat Dec 19, 2009 11:41 pm
Location: Sweden
Contact:

Re: SyncRowHeight - #7417

Post by mats » Wed Mar 04, 2020 9:58 am

Ok we'll need a test case to reproduce. If this is not about syncRowHeight issue, please start a new thread and try to provide a test case then we'll debug it.
Tired of debugging javascript errors in web applications? Try our new error logging service RootCause, or read more on the Sencha blog

@bryntum
Facebook
API documentation

User avatar
bee
Posts: 9
Joined: Mon Oct 09, 2017 3:29 pm

Re: SyncRowHeight - #7417

Post by bee » Wed Mar 04, 2020 10:02 am

Since the bug only encountered after the version update i cant say for sure if it is related or not but looking at the stacktrace of the error it sure looks as if its related to scrolling (and i dont know how the sources changed in terms of scrolling and updating the view)
Last edited by bee on Wed Mar 04, 2020 10:05 am, edited 1 time in total.

User avatar
mats
Core Developer
Core Developer
Posts: 15590
Joined: Sat Dec 19, 2009 11:41 pm
Location: Sweden
Contact:

Re: SyncRowHeight - #7417

Post by mats » Wed Mar 04, 2020 10:04 am

Ok, please start a new thread and include all the details of our version, Ext version + browser + test case and we'll help!
Tired of debugging javascript errors in web applications? Try our new error logging service RootCause, or read more on the Sencha blog

@bryntum
Facebook
API documentation

Post Reply