fix: fix undo and redo actions

src/controller/menuBar.ts

@@ -93,11 +93,17 @@ export class MenuBarController
     };
 
     public undo = () => {
-        this.monacoService.commandService.executeCommand(ACTION_QUICK_UNDO);
+        this.monacoService.commandService.executeCommand(
+            ACTION_QUICK_UNDO,
+            this.focusinEle
+        );
     };
 
     public redo = () => {
-        this.monacoService.commandService.executeCommand(ACTION_QUICK_REDO);
+        this.monacoService.commandService.executeCommand(
+            ACTION_QUICK_REDO,
+            this.focusinEle
+        );
     };
 
     public gotoQuickCommand = () => {

src/monaco/quickRedo.ts

@@ -1,6 +1,6 @@
 import 'reflect-metadata';
 import { localize } from 'mo/i18n/localize';
-import { KeyMod, KeyCode } from 'mo/monaco';
+import { KeyMod, KeyCode, Uri, editor as MonacoEditor } from 'mo/monaco';
 import { EditorService, IEditorService } from 'mo/services';
 import { container } from 'tsyringe';
 import { Action2, KeybindingWeight } from './common';
@@ -32,11 +32,37 @@ export class QuickRedo extends Action2 {
         this.editorService = container.resolve(EditorService);
     }
 
-    run() {
-        const editorInstance = this.editorService.editorInstance;
-        editorInstance!.focus();
-        if (!document.execCommand(QuickRedo.DESC)) {
-            editorInstance?.getModel()?.[QuickRedo.DESC]?.();
+    isTextdom(ele: Element): ele is HTMLInputElement {
+        return typeof (ele as HTMLInputElement).selectionStart === 'number';
+    }
+
+    run(accessor, ...args) {
+        const focusinEle = args[0];
+        const currentFocusinEle: Element | null =
+            focusinEle || document.activeElement;
+        if (
+            currentFocusinEle &&
+            this.isTextdom(currentFocusinEle) &&
+            !currentFocusinEle.className.includes('monaco')
+        ) {
+            // native dom use the native methods
+            document.execCommand('redo');
+        } else {
+            // monaco component should use the method from instance
+            const editorInstance = this.editorService.editorInstance;
+            if (editorInstance) {
+                const currentActiveGroup = this.editorService.getState()
+                    .current;
+                if (currentActiveGroup) {
+                    const tab = this.editorService.getTabById(
+                        currentActiveGroup.activeTab!,
+                        currentActiveGroup
+                    );
+                    editorInstance?.focus();
+                    const model = MonacoEditor.getModel(Uri.parse(tab!.id!))!;
+                    (model as any).redo();
+                }
+            }
         }
     }
 }

src/monaco/quickUndo.ts

@@ -1,6 +1,6 @@
 import 'reflect-metadata';
 import { localize } from 'mo/i18n/localize';
-import { KeyMod, KeyCode } from 'mo/monaco';
+import { KeyMod, KeyCode, Uri, editor as MonacoEditor } from 'mo/monaco';
 import { EditorService, IEditorService } from 'mo/services';
 import { container } from 'tsyringe';
 import { Action2, KeybindingWeight } from './common';
@@ -32,11 +32,37 @@ export class QuickUndo extends Action2 {
         this.editorService = container.resolve(EditorService);
     }
 
-    run() {
-        const editorInstance = this.editorService.editorInstance;
-        editorInstance!.focus();
-        if (!document.execCommand(QuickUndo.DESC)) {
-            editorInstance?.getModel()?.[QuickUndo.DESC]?.();
+    isTextdom(ele: Element): ele is HTMLInputElement {
+        return typeof (ele as HTMLInputElement).selectionStart === 'number';
+    }
+
+    run(accessor, ...args) {
+        const focusinEle = args[0];
+        const currentFocusinEle: Element | null =
+            focusinEle || document.activeElement;
+        if (
+            currentFocusinEle &&
+            this.isTextdom(currentFocusinEle) &&
+            !currentFocusinEle.className.includes('monaco')
+        ) {
+            // native dom use the native methods
+            document.execCommand('undo');
+        } else {
+            // monaco component should use the method from instance
+            const editorInstance = this.editorService.editorInstance;
+            if (editorInstance) {
+                const currentActiveGroup = this.editorService.getState()
+                    .current;
+                if (currentActiveGroup) {
+                    const tab = this.editorService.getTabById(
+                        currentActiveGroup.activeTab!,
+                        currentActiveGroup
+                    );
+                    editorInstance?.focus();
+                    const model = MonacoEditor.getModel(Uri.parse(tab!.id!))!;
+                    (model as any).undo();
+                }
+            }
         }
     }
 }