Click on column header itself also triggers re-order of the column

Caleydo · Jul 28, 2016 · 19ead89 · 19ead89
1 parent bb8fe70
commit 19ead89
Showing 1 changed file with 7 additions and 1 deletion.
diff --git a/src/ui.ts b/src/ui.ts
@@ -517,8 +517,14 @@ export class HeaderRenderer {
     var $headers = $base.selectAll('div.' + clazz).data(columns, (d) => d.id);
     var $headers_enter = $headers.enter().append('div').attr({
       'class': clazz
+    })
+    .on('click', (d) => {
+      if (this.options.manipulative && !d3.event.defaultPrevented && d3.event.currentTarget === d3.event.target) {
+        d.toggleMySorting();
+      }
     });
-    var $header_enter_div = $headers_enter.append('div').classed('lu-label', true).on('click', (d) => {
+    var $header_enter_div = $headers_enter.append('div').classed('lu-label', true)
+      .on('click', (d) => {
         if (this.options.manipulative && !d3.event.defaultPrevented) {
           d.toggleMySorting();
         }