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

arrangeComplete fire too many event's after you press multiple filters within 1 second. #1442

Open
mmativ opened this issue Sep 27, 2018 · 3 comments
Labels

Comments

@mmativ
Copy link

mmativ commented Sep 27, 2018

Test case: https://codepen.io/desandro/pen/pvXxRN?editors=0001

Try to press metal and transition filter button like 4 times in 1 second and open the console, you can notice there's many events happen, then after try to press 1 filter and check the console, there's couple of event's even you press 1 filter only.

@desandro desandro added the bug label Sep 27, 2018
@desandro
Copy link
Member

Thank you for reporting this issue. This is definitely a bug. I'll be looking into this in the coming months.

@chocolata
Copy link

Hi Dave, I'm also experiencing troubles with the arrangeComplete callback. In my setup, it fires unreliably, the first arrangements it never fires, then it fires 4-5 times in a row... Experiments with layoutComplete cause the same behavior.

The only workaround I've found is:

$grid.bind("transitionend webkitTransitionEnd oTransitionEnd MSTransitionEnd", function(){
	console.log("Wu-tang clan ain't nothing to f*ck with");
});

But this fires consistently a multitude of times...
I've also come across this issue but this seems to be outdated.

Can you kindly help us out? Thanks in advance.

@hirbod
Copy link

hirbod commented Dec 4, 2020

I can confirm this @desandro . If I switch filters quickly within a second, arrangeComplete fires 15 times until it stops, even though .arrange() got called only 3 times. My current workaround is to lock filter changes until arrangeComplete callback release it again but thats not a very snappy workaround.

Newest version, vanilla.

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

4 participants