Browse Source

Adding cookie support for toggle all columns options

djhvscf 5 years ago
parent
commit
6f842a870a
1 changed files with 5 additions and 8 deletions
  1. 5 8
      src/extensions/cookie/bootstrap-table-cookie.js

+ 5 - 8
src/extensions/cookie/bootstrap-table-cookie.js

@@ -373,16 +373,13 @@ $.BootstrapTable = class extends $.BootstrapTable {
 
   _toggleColumn (...args) {
     super._toggleColumn(...args)
+    UtilsCookie.setCookie(this, UtilsCookie.cookieIds.columns, JSON.stringify(this.getVisibleColumns()))
+  }
 
-    const visibleColumns = []
-
-    for (const column of this.columns) {
-      if (column.visible) {
-        visibleColumns.push(column.field)
-      }
-    }
+  _toggleAllColumns (...args) {
+    super._toggleAllColumns(...args)
 
-    UtilsCookie.setCookie(this, UtilsCookie.cookieIds.columns, JSON.stringify(visibleColumns))
+    UtilsCookie.setCookie(this, UtilsCookie.cookieIds.columns, JSON.stringify(this.getVisibleColumns()))
   }
 
   selectPage (page) {