Merge pull request #242 from twolff-iow/master

adding size-changed event

Author: danomatic

src/angular-gridster.js

@@ -1838,8 +1838,8 @@
 	 * @param GridsterDraggable
 	 * @param GridsterResizable
 	 */
-	.directive('gridsterItem', ['$parse', 'GridsterDraggable', 'GridsterResizable',
-		function($parse, GridsterDraggable, GridsterResizable) {
+	.directive('gridsterItem', ['$parse', '$rootScope', 'GridsterDraggable', 'GridsterResizable',
+		function($parse, $rootScope, GridsterDraggable, GridsterResizable) {
 			return {
 				restrict: 'EA',
 				controller: 'GridsterItemCtrl',
@@ -1958,6 +1958,8 @@
 							item.gridster.moveOverlappingItems(item);
 							gridster.layoutChanged();
 						}
+
+						$rootScope.$broadcast('gridster-item-size-changed', item.sizeX, item.sizeY);
 					}
 					scope.$watch(function() {
 						return item.sizeY + ',' + item.sizeX + ',' + item.minSizeX + ',' + item.maxSizeX + ',' + item.minSizeY + ',' + item.maxSizeY;

test/spec/gridster-directive.js

@@ -10,6 +10,8 @@ describe('gridster directive', function() {
 	var startCount;
 	var resizeCount;
 	var stopCount;
+	var rootScope;
+	var broadcastOnRootScope;
 
 	var dragHelper = function(el, dx, dy) {
 		el.simulate('mouseover').simulate('drag', {
@@ -20,6 +22,9 @@ describe('gridster directive', function() {
 	};
 
 	beforeEach(inject(function($rootScope, $compile) {
+		rootScope = $rootScope;
+		broadcastOnRootScope = spyOn(rootScope, '$broadcast').and.callThrough();
+
 		$scope = $rootScope.$new();
 		startCount = resizeCount = stopCount = 0;
 
@@ -89,31 +94,49 @@ describe('gridster directive', function() {
 
 	it('should initialize resizable', function() {
 		var $widget = $el.find('li:first-child');
-		setTimeout(function() {
-			expect($widget.find('.handle-n').length).toBe(1);
-		});
+
+		expect($widget.find('.handle-n').length).toBe(1);
 	});
 
 	it('should update widget dimensions on resize & trigger custom resize events', function() {
+		var $widget = $el.find('li:first-child');
+		var handle = $widget.find('.handle-e');
 
-		setTimeout(function() {
-			var $widget = $el.find('li:first-child');
-			var handle = $widget.find('.handle-e');
+		expect($widget.width()).toBe(155);
+		expect($scope.dashboard.widgets[0].sizeX).toBe(1);
+		expect(startCount).toBe(0);
+		expect(resizeCount).toBe(0);
+		expect(stopCount).toBe(0);
 
-			expect($widget.width()).toBe(155);
-			expect($scope.dashboard.widgets[0].sizeX).toBe(1);
-			expect(startCount).toBe(0);
-			expect(resizeCount).toBe(0);
-			expect(stopCount).toBe(0);
+		dragHelper(handle, 50); // should resize to next width step
 
-			dragHelper(handle, 50); // should resize to next width step
+		expect($widget.width()).toBe(320);
+		expect($scope.dashboard.widgets[0].sizeX).toBe(2);
+		expect(startCount).toBe(1);
+		expect(resizeCount).toBe(1);
+		expect(stopCount).toBe(1);
+	});
 
-			expect($widget.width()).toBe(320);
-			expect($scope.dashboard.widgets[0].sizeX).toBe(2);
-			expect(startCount).toBe(1);
-			expect(resizeCount).toBe(1);
-			expect(stopCount).toBe(1);
-		});
+	it('should broadcast "size changed" event on resize', function() {
+		// arrange
+		var eHandle = $el.find('li:first-child').find('.handle-e');
+		var sHandle = $el.find('li:first-child').find('.handle-s');
+		broadcastOnRootScope.calls.reset();
+
+		// act
+		dragHelper(eHandle, 50);
+
+		// assert
+		expect(broadcastOnRootScope).toHaveBeenCalledWith('gridster-item-size-changed', 2, 1);
+
+		// arrange
+		broadcastOnRootScope.calls.reset();
+
+		// act
+		dragHelper(sHandle, 0, 50);
+
+		// assert
+		expect(broadcastOnRootScope).toHaveBeenCalledWith('gridster-item-size-changed', 2, 2);
 	});
 
 });