Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Cannot read property 'length' of undefined #403

Closed
MorelSerge opened this issue Jul 13, 2020 · 4 comments
Closed

Cannot read property 'length' of undefined #403

MorelSerge opened this issue Jul 13, 2020 · 4 comments
Labels

Comments

@MorelSerge
Copy link

MorelSerge commented Jul 13, 2020

I'm stumbling on an error: Cannot read property 'length' of undefined

It occurs at dist/muuri.module.js in _updateDragDirection at line 2241:21, meaning right here:

if (dragPositions.length) {

Any help is appreciated!

@niklasramo
Copy link
Collaborator

Hi @Vluf ! Can you give specific steps how to reproduce this error? A reduced test case would be nice.

@MorelSerge
Copy link
Author

Hi @niklasramo, thanks for getting back to me!
Unfortunately I cannot reproduce this error, it is merely happening in our production SPA and only on Android mobile.

This issue happens during dragging tiles. We happen to log the dragging, so right before crashing it's still logging the drag events.
E.g.

[{"clientX":269,"clientY":496.6666870117187,"screenX":322.3333435058594,"isFirst":false,"distance":0,"deltaX":0,"deltaY":0,"isFinal":false,"deltaTime":0,"srcEvent":"[object TouchEvent]","pageX":322.3333435058594,"pageY":340,"pointerType":"touch","screenY":421,"identifier":0,"type":"move","target":"[object HTMLDivElement]"}]

I'm more than happy to investigate further or deploy some logging for this, but as of right now I cannot reproduce.

Any guidance in how to proceed from here?
Thanks!

@niklasramo
Copy link
Collaborator

@Vluf I think I fixed this in the latest version v0.9.2, let me know if you still encounter this issue with that version.

@MorelSerge
Copy link
Author

@niklasramo According to Sentry, this is indeed fixed in v0.9.2. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

No branches or pull requests

2 participants