Added checkbox in dist_bar viz to enable sorting of bars based on x a…

…xis labels (#1379) * Added order bars option for dist_bar viz * Make order_bar checkbox to cover half the colum and translate the description
apache · Oct 28, 2016 · d2826ab · d2826ab
1 parent 6ab769f
commit d2826ab
Show file tree

Hide file tree

Showing 3 changed files with 17 additions and 2 deletions.
diff --git a/caravel/assets/visualizations/nvd3_vis.js b/caravel/assets/visualizations/nvd3_vis.js
@@ -159,7 +159,17 @@ function nvd3Vis(slice) {
 
             stacked = fd.bar_stacked;
             chart.stacked(stacked);
-
+            if (fd.order_bars) {
+              payload.data.forEach((d) => {
+                d.values.sort(
+                  function compare(a, b) {
+                    if (a.x < b.x) return -1;
+                    if (a.x > b.x) return 1;
+                    return 0;
+                  }
+                );
+              });
+            }
             if (fd.show_bar_value) {
               setTimeout(function () {
                 addTotalBarValues(chart, payload.data, stacked);

diff --git a/caravel/forms.py b/caravel/forms.py
@@ -250,6 +250,11 @@ def __init__(self, viz):
                 "default": False,
                 "description": "Show the value on top of the bars or not"
             }),
+            'order_bars': (BetterBooleanField, {
+                "label": _("Sort Bars"),
+                "default": False,
+                "description": _("Sort bars by x labels."),
+            }),
             'show_controls': (BetterBooleanField, {
                 "label": _("Extra Controls"),
                 "default": False,

diff --git a/caravel/viz.py b/caravel/viz.py
@@ -1320,7 +1320,7 @@ class DistributionBarViz(DistributionPieViz):
             ('y_axis_format', 'bottom_margin'),
             ('x_axis_label', 'y_axis_label'),
             ('reduce_x_ticks', 'contribution'),
-            ('show_controls', None),
+            ('show_controls', 'order_bars'),
         )
     },)
     form_overrides = {