Updates based on PR #1

Linechart suffered from bad copy pasta. Thanks @jclemans

Updated the concatenated and minified files.

Bumped version.

Author: scarroll

angular-graphael.js

@@ -46,7 +46,7 @@ angular.module('angular-graphael').directive('barchart', ["$window", "mapData",
                     y = scope.y || 100,
                     // Width
                     width = scope.width || 100,
-                    // height
+                    // Height
                     height = scope.height || 100;
 
                 // If you don't remove the old chart, you're gonna have a bad time.
@@ -74,8 +74,8 @@ angular.module('angular-graphael').directive('dotchart', ["$window", "mapData",
     return {
         restrict: 'E',
         scope: {
-            valuesx: '=',
-            valuesy: '=',
+            valuesX: '=',
+            valuesY: '=',
             size: '=',
             options: '=',
             x: '=',
@@ -86,17 +86,21 @@ angular.module('angular-graphael').directive('dotchart', ["$window", "mapData",
             // Right now we only care about watching if the data changes.
             scope.$watch('size', function () {
                 var r,
+                // X coordinate
                     x = scope.x || 0,
+                // Y coordinate
                     y = scope.y || 0,
+                // Width
                     width = scope.width || 500,
+                // Height
                     height = scope.height || 80;
 
                 // If you don't remove the old chart, you're gonna have a bad time.
                 element[0].innerHTML = '';
                 // Set up the canvas
                 r = $window.Raphael(element[0]);
                 // Add the chart to the canvas with all of our options and data.
-                r.dotchart(x, y, width, height, scope.valuesx, scope.valuesy, mapData(scope.size), scope.options);
+                r.dotchart(x, y, width, height, scope.valuesX, scope.valuesY, mapData(scope.size), scope.options);
             });
         }
     };
@@ -121,8 +125,8 @@ angular.module('angular-graphael').directive('linechart', ["$window", "mapData",
             x: '=',
             y: '=',
             options: '=',
-            valuesx: '=',
-            valuesy: '='
+            valuesX: '=',
+            valuesY: '='
         },
         template: '<div></div>',
         link: function (scope, element) {
@@ -135,15 +139,15 @@ angular.module('angular-graphael').directive('linechart', ["$window", "mapData",
                     y = scope.y || 100,
                 // Width
                     width = scope.width || 100,
-                // height
+                // Height
                     height = scope.height || 100;
 
                 // If you don't remove the old chart, you're gonna have a bad time.
                 element[0].innerHTML = '';
                 // Set up the canvas
                 r = $window.Raphael(element[0]);
                 // Add the chart to the canvas with all of our options and data.
-                r.barchart(x, y, width, height, mapData(scope.valuesx), mapData(scope.valuesy), scope.options);
+                r.linechart(x, y, width, height, mapData(scope.valuesX), mapData(scope.valuesY), scope.options);
             });
         }
     };

angular-graphael.min.js

@@ -1,6 +1,6 @@
 {
   "name": "angular-graphael",
-  "version": "0.0.1",
+  "version": "0.1.0",
   "description": "A set of AngularJS directives to make charts using gRaphaël.",
   "homepage": "https://github.com/scottux/angular-graphael",
   "authors": [

bower.json

@@ -1,6 +1,6 @@
 {
   "name": "angular-graphael",
-  "version": "0.0.1",
+  "version": "0.1.0",
   "description": "A set of AngularJS directives to make charts using gRaphaël.",
   "private": true,
   "devDependencies": {

package.json

@@ -30,7 +30,7 @@ angular.module('angular-graphael').directive('barchart', function ($window, mapD
                     y = scope.y || 100,
                     // Width
                     width = scope.width || 100,
-                    // height
+                    // Height
                     height = scope.height || 100;
 
                 // If you don't remove the old chart, you're gonna have a bad time.

src/directives/barchartDirective.js

@@ -24,9 +24,13 @@ angular.module('angular-graphael').directive('dotchart', function ($window, mapD
             // Right now we only care about watching if the data changes.
             scope.$watch('size', function () {
                 var r,
+                // X coordinate
                     x = scope.x || 0,
+                // Y coordinate
                     y = scope.y || 0,
+                // Width
                     width = scope.width || 500,
+                // Height
                     height = scope.height || 80;
 
                 // If you don't remove the old chart, you're gonna have a bad time.

src/directives/dotchartDirective.js

@@ -31,15 +31,15 @@ angular.module('angular-graphael').directive('linechart', function ($window, map
                     y = scope.y || 100,
                 // Width
                     width = scope.width || 100,
-                // height
+                // Height
                     height = scope.height || 100;
 
                 // If you don't remove the old chart, you're gonna have a bad time.
                 element[0].innerHTML = '';
                 // Set up the canvas
                 r = $window.Raphael(element[0]);
                 // Add the chart to the canvas with all of our options and data.
-                r.barchart(x, y, width, height, mapData(scope.valuesX), mapData(scope.valuesY), scope.options);
+                r.linechart(x, y, width, height, mapData(scope.valuesX), mapData(scope.valuesY), scope.options);
             });
         }
     };