Make column toggle use entire li for clicks instead of checkbox. (#243)

aslagle · Jul 5, 2015 · d2434fe · d2434fe
1 parent 0dfd4d3
commit d2434fe
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/lib/reactive_table.js b/lib/reactive_table.js
@@ -510,10 +510,10 @@ Template.reactiveTable.events({
         getUpdateHandleForTemplate(template)(template.context);
     },
 
-    'change .reactive-table-columns-dropdown input': function (event) {
+    'click .reactive-table-columns-dropdown li': function (event) {
         var template = Template.instance();
-        var target = $(event.target);
-        var fieldId = target.attr('data-fieldid');
+        var target = $(event.currentTarget);
+        var fieldId = target.find('input').attr('data-fieldid');
         var visibleFields = template.context.visibleFields.get();
         var visibleField = _.findWhere(visibleFields, {fieldId: fieldId});
         if (visibleField) {

diff --git a/package.js b/package.js
@@ -1,6 +1,6 @@
 Package.describe({
   summary: "A reactive table designed for Meteor",
-  version: "0.8.4",
+  version: "0.8.5",
   name: "aslagle:reactive-table",
   git: "https://github.com/aslagle/reactive-table.git"
 });