fix event propagation order

olifolkerd · Jan 19, 2025 · cd66586 · cd66586
1 parent 48ee66a
commit cd66586
Show file tree

Hide file tree

Showing 5 changed files with 29 additions and 8 deletions.
diff --git a/dist/js/tabulator_esm.js b/dist/js/tabulator_esm.js
@@ -27201,8 +27201,9 @@ class InteractionManager extends CoreFeature {
 		var keys = Object.keys(targets).reverse(),
 		listener = this.listeners[type],
 		matches = {},
+		output = {},
 		targetMatches = {};
-		
+
 		for(let key of keys){
 			let component,
 			target = targets[key],
@@ -27257,8 +27258,14 @@ class InteractionManager extends CoreFeature {
 		}
 
 		this.previousTargets = targetMatches;
+
+		//reverse order keys are set in so events trigger in correct sequence
+		Object.keys(targets).forEach((key) => {
+			let value = matches[key];
+			output[key] = value;
+		});
 
-		return matches;
+		return output;
 	}
 
 	triggerEvents(type, e, targets){

diff --git a/dist/js/tabulator_esm.js.map b/dist/js/tabulator_esm.js.map
diff --git a/dist/js/tabulator_esm.mjs b/dist/js/tabulator_esm.mjs
@@ -27201,8 +27201,9 @@ class InteractionManager extends CoreFeature {
 		var keys = Object.keys(targets).reverse(),
 		listener = this.listeners[type],
 		matches = {},
+		output = {},
 		targetMatches = {};
-		
+
 		for(let key of keys){
 			let component,
 			target = targets[key],
@@ -27257,8 +27258,14 @@ class InteractionManager extends CoreFeature {
 		}
 
 		this.previousTargets = targetMatches;
+
+		//reverse order keys are set in so events trigger in correct sequence
+		Object.keys(targets).forEach((key) => {
+			let value = matches[key];
+			output[key] = value;
+		});
 
-		return matches;
+		return output;
 	}
 
 	triggerEvents(type, e, targets){

diff --git a/dist/js/tabulator_esm.mjs.map b/dist/js/tabulator_esm.mjs.map
diff --git a/src/js/core/tools/InteractionMonitor.js b/src/js/core/tools/InteractionMonitor.js
@@ -239,8 +239,9 @@ export default class InteractionManager extends CoreFeature {
 		var keys = Object.keys(targets).reverse(),
 		listener = this.listeners[type],
 		matches = {},
+		output = {},
 		targetMatches = {};
-		
+
 		for(let key of keys){
 			let component,
 			target = targets[key],
@@ -295,8 +296,14 @@ export default class InteractionManager extends CoreFeature {
 		}
 
 		this.previousTargets = targetMatches;
+
+		//reverse order keys are set in so events trigger in correct sequence
+		Object.keys(targets).forEach((key) => {
+			let value = matches[key];
+			output[key] = value;
+		});
 
-		return matches;
+		return output;
 	}
 
 	triggerEvents(type, e, targets){