$timeout on resize() (gridster.js line 550) about angular-gridster HOT 5 CLOSED

scope.$apply();
change to
$scope.$$phase || scope.$apply();

although this would solve your issue. i think generally $scope.$apply() should be very specific

from angular-gridster.

After doing some reading, the $timeout (with a true parameter) creates a deferred and then calls $scope.$apply. While not being as semantic it is more "zen" as it does not rely on the undocumented $$phase value.

That said, I understand the $timeout is a little ugly and that $$phase should not be deprecated until Angular 2.0(ish) so I could go either way on this issue.

from angular-gridster.

I think the issue is that resize is sometimes called within an angular cycle and sometimes from a non-angular event. That can probably be fixed in the code that calls resize, rather than making resize do timeouts and other special code to handle the different usages.

from angular-gridster.

Sounds 10x better; I definitely will read through/understand the source better.

from angular-gridster.

Fix just pushed. Closing for now. Please report any regressions.

from angular-gridster.