fix(app): column width is an integer

passy · passy · commit f04d3a2e1369 · 2013-07-29T10:47:22.000+02:00
The columnWidth option has to be converted to an integer work with masonry.
diff --git a/src/angular-masonry.js b/src/angular-masonry.js
@@ -104,7 +104,7 @@
           var attrOptions = scope.$eval(attrs.options);
           var options = angular.extend(attrOptions || {}, {
             itemSelector: attrs.itemSelector || '.masonry-brick',
-            columnWidth: attrs.columnWidth
+            columnWidth: parseInt(attrs.columnWidth, 10)
           });
           element.masonry(options);
 
diff --git a/test/spec/directive.coffee b/test/spec/directive.coffee
@@ -31,7 +31,7 @@ describe 'angular-masonry', ->
 
     expect($.fn.masonry).toHaveBeenCalledOnce()
     call = $.fn.masonry.firstCall
-    expect(call.args[0].columnWidth).toBe '200'
+    expect(call.args[0].columnWidth).toBe 200
   )
 
   it 'should pass on the item-selector attribute', inject(($compile) =>

Original file line number	Diff line number	Diff line change
`@@ -31,7 +31,7 @@ describe 'angular-masonry', ->`
`31`	`31`
`32`	`32`	`expect($.fn.masonry).toHaveBeenCalledOnce()`
`33`	`33`	`call = $.fn.masonry.firstCall`
`34`		`- expect(call.args[0].columnWidth).toBe '200'`
	`34`	`+ expect(call.args[0].columnWidth).toBe 200`
`35`	`35`	`)`
`36`	`36`
`37`	`37`	`it 'should pass on the item-selector attribute', inject(($compile) =>`