Improve chart features

[grtjn]

Move code from landing control to charts service, and allow callback:

-    service.top10Chart = function(title, type, xFacet, xLabel, limit) {
+    service.top10Chart = function(title, type, xFacet, xLabel, limit, callback) {
       return {
         options: {
           chart: {
@@ -30,6 +30,21 @@
           },
           legend: {
             enabled: false
+          },
+          plotOptions: {
+            series: {
+              cursor: 'pointer',
+              point: {
+                events: {
+                  click: function (e) {
+                    var facet = this.series.name;
+                    var x = this.xCategory || this.x;
+                    var y = this.yCategory || this.y;
+                    callback(facet, x, y);
+                  }
+                }
+              }
+            }

[janmichaelyu]

@grtjn Please assign to me. I think there are some improvements to be made as well as charts have multiple calls and are slow for big datasets.

[grtjn]

There should be one call per chart. It could potentially reuse facets from search, but the landing page is not showing search results, and facets wouldn't support co-occurrence charts.

But please, do go ahead. Consider this ticket yours (cannot assign it)..

[janmichaelyu]

Pull request that fixes the multiple calls:

https://github.com/ryanjdew/ml-highcharts-ng/pull/33

[janmichaelyu]

Pull request that moves the chart functions to a separate service: https://github.com/marklogic-community/slush-marklogic-node/pull/552