tabulator-tables 5.4.1 → 5.4.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "tabulator-tables",
-  "version": "5.4.1",
+  "version": "5.4.2",
   "description": "Interactive table generation JavaScript library",
   "style": "dist/css/tabulator.css",
   "main": "dist/js/tabulator.js",
@@ -53,7 +53,7 @@
     "eslint-plugin-only-warn": "^1.0.3",
     "fs-extra": "^9.0.1",
     "globby": "^11.0.2",
-    "node-sass": "^5.0.0",
+    "node-sass": "^7.0.3",
     "postcss": "^8.2.4",
     "postcss-prettify": "^0.3.4",
     "rollup": "^2.36.1",

package/src/js/core/ColumnManager.js

@@ -535,10 +535,11 @@ export default class ColumnManager extends CoreFeature {
 	
 	scrollToColumn(column, position, ifVisible){
 		var left = 0,
-		offset = 0,
+		offset = column.getLeftOffset(),
 		adjust = 0,
 		colEl = column.getElement();
 		
+		
 		return new Promise((resolve, reject) => {
 			
 			if(typeof position === "undefined"){
@@ -565,16 +566,13 @@ export default class ColumnManager extends CoreFeature {
 				
 				//check column visibility
 				if(!ifVisible){
-					
-					offset = colEl.offsetLeft;
-					
 					if(offset > 0 && offset + colEl.offsetWidth < this.element.clientWidth){
 						return false;
 					}
 				}
 				
 				//calculate scroll position
-				left = colEl.offsetLeft + adjust;
+				left = offset + adjust;
 				
 				left = Math.max(Math.min(left, this.table.rowManager.element.scrollWidth - this.table.rowManager.element.clientWidth),0);
 				

package/src/js/core/column/Column.js

@@ -721,6 +721,16 @@ class Column extends CoreFeature{
 		return width;
 	}
 
+	getLeftOffset(){
+		var offset = this.element.offsetLeft;
+
+		if(this.parent.isGroup){
+			offset += this.parent.getLeftOffset();
+		}
+
+		return offset;
+	}
+
 	getHeight(){
 		return Math.ceil(this.element.getBoundingClientRect().height);
 	}

package/src/js/modules/ColumnCalcs/ColumnCalcs.js

@@ -250,6 +250,16 @@ class ColumnCalcs extends Module{
 		}
 	}
 	
+	reinitializeCalcs(){
+		if(this.topCalcs.length){
+			this.initializeTopRow();
+		}
+
+		if(this.botCalcs.length){
+			this.initializeBottomRow();
+		}
+	}
+	
 	initializeTopRow(){
 		if(!this.topInitialized){
 			this.table.columnManager.getContentsElement().insertBefore(this.topElement, this.table.columnManager.headersElement.nextSibling);

package/src/js/modules/GroupRows/Group.js

@@ -38,17 +38,19 @@ class Group{
 		this.createValueGroups();
 	}
 	
-	wipe(){
-		if(this.groupList.length){
-			this.groupList.forEach(function(group){
-				group.wipe();
-			});
-		}else{
-			this.rows.forEach((row) => {
-				if(row.modules){
-					delete row.modules.group;
-				}
-			});
+	wipe(elementsOnly){
+		if(!elementsOnly){
+			if(this.groupList.length){
+				this.groupList.forEach(function(group){
+					group.wipe();
+				});
+			}else{
+				this.rows.forEach((row) => {
+					if(row.modules){
+						delete row.modules.group;
+					}
+				});
+			}
 		}
 		
 		this.element = false;

package/src/js/modules/GroupRows/GroupRows.js

@@ -92,7 +92,7 @@ class GroupRows extends Module{
 			
 			this.groupIDLookups = [];
 			
-			if(Array.isArray(groupBy)){
+			if(groupBy){
 				if(this.table.modExists("columnCalcs") && this.table.options.columnCalcs != "table" && this.table.options.columnCalcs != "both"){
 					this.table.modules.columnCalcs.removeCalcs();
 				}
@@ -233,6 +233,10 @@ class GroupRows extends Module{
 		}
 		
 		this.configureGroupSetup();
+
+		if(!groups && this.table.modExists("columnCalcs") && this.table.options.columnCalcs === true){
+			this.table.modules.columnCalcs.reinitializeCalcs();
+		}
 		
 		this.refreshData();
 		
@@ -479,7 +483,7 @@ class GroupRows extends Module{
 		}
 		
 		Object.values(oldGroups).forEach((group) => {
-			group.wipe();
+			group.wipe(true);
 		});	
 	}
 	

package/src/js/modules/MoveColumns/MoveColumns.js

@@ -50,7 +50,7 @@ class MoveColumns extends Module{
 			
 			config.mousemove = function(e){
 				if(column.parent === self.moving.parent){
-					if((((self.touchMove ? e.touches[0].pageX : e.pageX) - Helpers.elOffset(colEl).left) + self.table.columnManager.element.scrollLeft) > (column.getWidth() / 2)){
+					if((((self.touchMove ? e.touches[0].pageX : e.pageX) - Helpers.elOffset(colEl).left) + self.table.columnManager.contentsElement.scrollLeft) > (column.getWidth() / 2)){
 						if(self.toCol !== column || !self.toColAfter){
 							colEl.parentNode.insertBefore(self.placeholderElement, colEl.nextSibling);
 							self.moveColumn(column, true);