refactor

Author: azmy60

src/js/modules/Edit/Edit.js

@@ -76,6 +76,10 @@ export default class Edit extends Module{
 		// this.subscribe("keybinding-nav-right", this.navigateRight.bind(this, undefined));
 		this.subscribe("keybinding-nav-up", this.navigateUp.bind(this, undefined));
 		this.subscribe("keybinding-nav-down", this.navigateDown.bind(this, undefined));
+		
+		// Add event handlers for other modules to access editing state and functionality
+		this.subscribe("edit-check-editing", this.checkEditing.bind(this));
+		this.subscribe("edit-cancel-cell", this.cancelEditEvent.bind(this));
 
 		if(Object.keys(this.table.options).includes("editorEmptyValue")){
 			this.convertEmptyValues = true;
@@ -449,6 +453,19 @@ export default class Edit extends Module{
 		return this.currentCell ? this.currentCell.getComponent() : false;
 	}
 
+	checkEditing(){
+		return !!this.currentCell;
+	}
+	
+	cancelEditEvent(){
+		if(this.currentCell){
+			this.cancelEdit();
+			return true;
+		}
+		return false;
+	}
+	
+	
 	clearEditor(cancel){
 		var cell = this.currentCell,
 		cellEl;

src/js/modules/SelectRange/SelectRange.js

@@ -404,13 +404,16 @@ export default class SelectRange extends Module {
 	///////////////////////////////////
 
 	keyNavigate(dir, e){
-		if(this.options("selectableRangeBlurEditOnNavigate") && this.table.modules.edit && this.table.modules.edit.currentCell){
-			if(dir === 'next' || dir === 'prev'){
-				// Cancel edit and move to the next cell if editing
-				this.table.modules.edit.currentCell.getComponent().cancelEdit();
-			}else{
-				// Prevent navigating while editing except for next/prev
-				return false;
+		if(this.options("selectableRangeBlurEditOnNavigate")){
+			const isEditing = this.chain("edit-check-editing");
+			
+			if(isEditing){
+				if(dir === 'next' || dir === 'prev'){
+					this.dispatch("edit-cancel-cell");
+				}else{
+					// Prevent navigating while editing except for next/prev
+					return false;
+				}
 			}
 		}