Fixed issue with showing/hiding columns

Iodine- · Iodine- · commit f7a137520924 · 2016-08-10T11:38:32.000-07:00
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,5 +1,10 @@
 # Changelog
 
+## 1.4.2
+
+### Bug Fixes
+- Fixed bug with showing/hiding columns
+
 ## 1.4.1
 
 ### Bug Fixes
diff --git a/dist/jquery.bootgrid.js b/dist/jquery.bootgrid.js
@@ -361,6 +361,7 @@ function renderColumnSelection(actions) {
 						var $this = $(this),
 							checkbox = $this.find(checkboxSelector);
 						if (!checkbox.prop("disabled")) {
+							column.visible = checkbox.prop("checked");
 							var enable = that.columns.where(isVisible).length > 1;
 							$this.parents(itemsSelector).find(selector + ":has(" + checkboxSelector + ":checked)")
 								._bgEnableAria(enable).find(checkboxSelector)._bgEnableField(enable);
@@ -372,7 +373,6 @@ function renderColumnSelection(actions) {
 					})
 					.on("change" + namespace, checkboxSelector, function(e){
 						var $this = $(this);
-						column.visible = $this.prop("checked");
 						that.element.trigger('toggleColumn', [column.id, column.text, column.visible]);
 					});
 				dropDown.find(getCssSelector(css.dropDownMenuItems)).append(item);
diff --git a/src/internal.js b/src/internal.js
@@ -351,6 +351,7 @@ function renderColumnSelection(actions) {
 						var $this = $(this),
 							checkbox = $this.find(checkboxSelector);
 						if (!checkbox.prop("disabled")) {
+							column.visible = checkbox.prop("checked");
 							var enable = that.columns.where(isVisible).length > 1;
 							$this.parents(itemsSelector).find(selector + ":has(" + checkboxSelector + ":checked)")
 								._bgEnableAria(enable).find(checkboxSelector)._bgEnableField(enable);
@@ -362,7 +363,6 @@ function renderColumnSelection(actions) {
 					})
 					.on("change" + namespace, checkboxSelector, function(e){
 						var $this = $(this);
-						column.visible = $this.prop("checked");
 						that.element.trigger('toggleColumn', [column.id, column.text, column.visible]);
 					});
 				dropDown.find(getCssSelector(css.dropDownMenuItems)).append(item);
