fix: scroll notes list when navigating with arrow keys

standardnotes · Jun 16, 2021 · 2473a78 · 2473a78
1 parent 91fc475
commit 2473a78
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 5 deletions.
diff --git a/app/assets/javascripts/views/editor/editor_view.ts b/app/assets/javascripts/views/editor/editor_view.ts
@@ -285,9 +285,6 @@ class EditorViewCtrl extends PureViewCtrl<unknown, EditorState> {
     if (note.dirty) {
       this.showSavingStatus();
     }
-    if (note.safeText().length === 0 && !showProtectedWarning) {
-      this.focusTitle();
-    }
   }
 
   async dismissProtectedWarning() {

diff --git a/app/assets/javascripts/views/notes/notes_view.ts b/app/assets/javascripts/views/notes/notes_view.ts
@@ -411,6 +411,8 @@ class NotesViewCtrl extends PureViewCtrl<unknown, NotesCtrlState> {
     await this.flushUI();
     await this.reloadNotes();
     await this.appState.noteTags.reloadTags();
+    const noteTitleEditorElement = document.getElementById('note-title-editor');
+    noteTitleEditorElement?.focus();
   }
 
   async handleTagChange(tag: SNTag) {
@@ -789,7 +791,10 @@ class NotesViewCtrl extends PureViewCtrl<unknown, NotesCtrlState> {
       return candidate.uuid === this.activeEditorNote?.uuid;
     });
     if (currentIndex + 1 < displayableNotes.length) {
-      this.selectNote(displayableNotes[currentIndex + 1]);
+      const nextNote = displayableNotes[currentIndex + 1];
+      this.selectNote(nextNote);
+      const nextNoteElement = document.getElementById(`note-${nextNote.uuid}`);
+      nextNoteElement?.focus();
     }
   }
 
@@ -806,7 +811,10 @@ class NotesViewCtrl extends PureViewCtrl<unknown, NotesCtrlState> {
     const displayableNotes = this.state.notes;
     const currentIndex = displayableNotes.indexOf(this.activeEditorNote!);
     if (currentIndex - 1 >= 0) {
-      this.selectNote(displayableNotes[currentIndex - 1]);
+      const previousNote = displayableNotes[currentIndex - 1];
+      this.selectNote(previousNote);
+      const previousNoteElement = document.getElementById(`note-${previousNote.uuid}`);
+      previousNoteElement?.focus();
       return true;
     } else {
       return false;