add keyboard shortcut for the edit button, #2036

src/public/app/services/root_command_executor.js

@@ -35,6 +35,13 @@ export default class RootCommandExecutor extends Component {
         }
     }
 
+    editReadOnlyNoteCommand() {
+        const noteContext = appContext.tabManager.getActiveContext();
+        noteContext.readOnlyTemporarilyDisabled = true;
+
+        appContext.triggerEvent("readOnlyTemporarilyDisabled", { noteContext });
+    }
+
     async cloneNoteIdsToCommand({noteIds}) {
         const d = await import("../dialogs/clone_to.js");
         d.showDialog(noteIds);

src/public/app/widgets/buttons/edit_button.js

@@ -24,8 +24,6 @@ export default class EditButton extends ButtonWidget {
         // can't do this in isEnabled() since isReadOnly is async
         this.toggleInt(await this.noteContext.isReadOnly());
 
-        console.log("await this.noteContext.isReadOnly()", await this.noteContext.isReadOnly());
-
         await super.refreshWithNote(note);
     }
 }

src/public/app/widgets/type_widgets/type_widget.js

@@ -41,15 +41,11 @@ export default class TypeWidget extends NoteContextAwareWidget {
 
     focus() {}
 
-    readOnlyTemporarilyDisabledEvent({noteContext}) {
+    async readOnlyTemporarilyDisabledEvent({noteContext}) {
         if (this.isNoteContext(noteContext.ntxId)) {
-            this.refresh();
+            await this.refresh();
-        }
-    }
 
-    readOnlyTemporarilyDisabledEvent({noteContext}) {
+            this.focus();
-        if (this.isNoteContext(noteContext.ntxId)) {
-            this.refresh();
         }
     }
 }

src/services/keyboard_actions.js

@@ -313,6 +313,12 @@ const DEFAULT_KEYBOARD_ACTIONS = [
         description: "Opens the dialog to include a note",
         scope: "text-detail"
     },
+    {
+        actionName: "editReadOnlyNote",
+        defaultShortcuts: [],
+        description: "Edit a read-only note",
+        scope: "window"
+    },
 
     {
         separator: "Attributes (labels & relations)"