Merge branch 'release/10.7.1'

app/assets/javascripts/services/ioService.ts

@@ -87,27 +87,37 @@ export class IOService {
     this.activeModifiers.delete(modifier);
   };
 
-  handleKeyDown = (event: KeyboardEvent): void => {
-    for (const modifier of this.modifiersForEvent(event)) {
-      this.addActiveModifier(modifier);
-    }
-    this.notifyObserver(event, KeyboardKeyEvent.Down);
-  };
-
-  handleComponentKeyDown = (modifier: KeyboardModifier | undefined): void => {
+  public handleComponentKeyDown = (
+    modifier: KeyboardModifier | undefined
+  ): void => {
     this.addActiveModifier(modifier);
   };
 
-  handleKeyUp = (event: KeyboardEvent): void => {
-    for (const modifier of this.modifiersForEvent(event)) {
-      this.removeActiveModifier(modifier);
-    }
+  public handleComponentKeyUp = (
+    modifier: KeyboardModifier | undefined
+  ): void => {
+    this.removeActiveModifier(modifier);
+  };
+
+  private handleKeyDown = (event: KeyboardEvent): void => {
+    this.updateAllModifiersFromEvent(event);
+    this.notifyObserver(event, KeyboardKeyEvent.Down);
+  };
+
+  private handleKeyUp = (event: KeyboardEvent): void => {
+    this.updateAllModifiersFromEvent(event);
     this.notifyObserver(event, KeyboardKeyEvent.Up);
   };
 
-  handleComponentKeyUp = (modifier: KeyboardModifier | undefined): void => {
-    this.removeActiveModifier(modifier);
-  };
+  private updateAllModifiersFromEvent(event: KeyboardEvent): void {
+    for (const modifier of Object.values(KeyboardModifier)) {
+      if (event.getModifierState(modifier)) {
+        this.addActiveModifier(modifier);
+      } else {
+        this.removeActiveModifier(modifier);
+      }
+    }
+  }
 
   handleWindowBlur = (): void => {
     for (const modifier of this.activeModifiers) {
@@ -136,7 +146,7 @@ export class IOService {
     return eventModifiers;
   }
 
-  eventMatchesKeyAndModifiers(
+  private eventMatchesKeyAndModifiers(
     event: KeyboardEvent,
     key: KeyboardKey | string,
     modifiers: KeyboardModifier[] = []

app/assets/javascripts/ui_models/app_state/notes_state.ts

@@ -114,42 +114,43 @@ export class NotesState {
 
   async selectNote(uuid: UuidString, userTriggered?: boolean): Promise<void> {
     const note = this.application.findItem(uuid) as SNNote;
+    if (!note) {
+      return;
+    }
 
     const hasMeta = this.io.activeModifiers.has(KeyboardModifier.Meta);
     const hasCtrl = this.io.activeModifiers.has(KeyboardModifier.Ctrl);
     const hasShift = this.io.activeModifiers.has(KeyboardModifier.Shift);
 
-    if (note) {
-      if (userTriggered && (hasMeta || hasCtrl)) {
-        if (this.selectedNotes[uuid]) {
-          delete this.selectedNotes[uuid];
-        } else if (await this.application.authorizeNoteAccess(note)) {
-          runInAction(() => {
-            this.selectedNotes[uuid] = note;
-            this.lastSelectedNote = note;
-          });
-        }
-      } else if (userTriggered && hasShift) {
-        await this.selectNotesRange(note);
-      } else {
-        const shouldSelectNote =
-          this.selectedNotesCount > 1 || !this.selectedNotes[uuid];
-        if (
-          shouldSelectNote &&
-          (await this.application.authorizeNoteAccess(note))
-        ) {
-          runInAction(() => {
-            this.selectedNotes = {
-              [note.uuid]: note,
-            };
-            this.lastSelectedNote = note;
-          });
-        }
+    if (userTriggered && (hasMeta || hasCtrl)) {
+      if (this.selectedNotes[uuid]) {
+        delete this.selectedNotes[uuid];
+      } else if (await this.application.authorizeNoteAccess(note)) {
+        runInAction(() => {
+          this.selectedNotes[uuid] = note;
+          this.lastSelectedNote = note;
+        });
       }
+    } else if (userTriggered && hasShift) {
+      await this.selectNotesRange(note);
+    } else {
+      const shouldSelectNote =
+        this.selectedNotesCount > 1 || !this.selectedNotes[uuid];
+      if (
+        shouldSelectNote &&
+        (await this.application.authorizeNoteAccess(note))
+      ) {
+        runInAction(() => {
+          this.selectedNotes = {
+            [note.uuid]: note,
+          };
+          this.lastSelectedNote = note;
+        });
+      }
+    }
 
-      if (this.selectedNotesCount === 1) {
-        await this.openNote(Object.keys(this.selectedNotes)[0]);
-      }
+    if (this.selectedNotesCount === 1) {
+      await this.openNote(Object.keys(this.selectedNotes)[0]);
     }
   }
 

app/assets/javascripts/ui_models/app_state/notes_view_state.ts

@@ -333,9 +333,14 @@ export class NotesViewState {
     if (this.isFiltering) {
       title = this.noteFilterText;
     }
+
     await this.appState.openNewNote(title);
-    this.reloadNotes();
-    this.appState.noteTags.reloadTags();
+    this.application.performFunctionWithAngularDigestCycleAfterAsyncChange(
+      () => {
+        this.reloadNotes();
+        this.appState.noteTags.reloadTags();
+      }
+    );
   };
 
   createPlaceholderNote = () => {

app/assets/javascripts/ui_models/application.ts

@@ -43,6 +43,7 @@ export class WebApplication extends SNApplication {
     platform: Platform,
     identifier: string,
     private $compile: angular.ICompileService,
+    private $timeout: angular.ITimeoutService,
     scope: angular.IScope,
     defaultSyncServerHost: string,
     public bridge: Bridge,
@@ -104,6 +105,16 @@ export class WebApplication extends SNApplication {
     this.webServices = services;
   }
 
+  /**
+   * If a UI change is made in an async function, Angular might not re-render the change.
+   * Use this function to force re-render the UI after an async function has made UI changes.
+   */
+  public performFunctionWithAngularDigestCycleAfterAsyncChange(
+    func: () => void
+  ) {
+    this.$timeout(func);
+  }
+
   public getAppState(): AppState {
     return this.webServices.appState;
   }

app/assets/javascripts/ui_models/application_group.ts

@@ -61,6 +61,7 @@ export class ApplicationGroup extends SNApplicationGroup {
       platform,
       descriptor.identifier,
       this.$compile,
+      this.$timeout,
       scope,
       this.defaultSyncServerHost,
       this.bridge,