refactor: AlertService

app/assets/javascripts/directives/views/accountMenu.ts

@@ -296,21 +296,16 @@ class AccountMenuCtrl extends PureViewCtrl {
     }
   }
 
-  mergeLocalChanged() {
+  async mergeLocalChanged() {
     if (!this.getState().formData.mergeLocal) {
-      this.application!.alertService!.confirm(
+      if (await confirmDialog({
-        STRING_ACCOUNT_MENU_UNCHECK_MERGE,
+        text: STRING_ACCOUNT_MENU_UNCHECK_MERGE,
-        undefined,
+        confirmButtonStyle: 'danger'
-        undefined,
+      })) {
-        undefined,
-        undefined,
-        () => {
         this.setFormDataState({
           mergeLocal: true
         });
-        },
+      }
-        true,
-      );
     }
   }
 
@@ -553,17 +548,12 @@ class AccountMenuCtrl extends PureViewCtrl {
       if (!signedIn) {
         message += STRING_REMOVE_PASSCODE_OFFLINE_ADDENDUM;
       }
-      this.application!.alertService!.confirm(
+      if (await confirmDialog({
-        message,
+        text: message,
-        undefined,
+        confirmButtonStyle: 'danger'
-        undefined,
+      })) {
-        undefined,
-        () => {
         this.application!.removePasscode();
-        },
+      }
-        undefined,
-        true,
-      );
     };
     const needsPrivilege = await this.application!.privilegesService!.actionRequiresPrivilege(
       ProtectedAction.ManagePasscode

app/assets/javascripts/services/alertService.ts

@@ -1,5 +1,5 @@
 /* eslint-disable prefer-promise-reject-errors */
-import { SNAlertService } from 'snjs';
+import { SNAlertService, ButtonType, DismissBlockingDialog } from 'snjs';
 import { SKAlert } from 'sn-stylekit';
 
 /** @returns a promise resolving to true if the user confirmed, false if they canceled */
@@ -41,55 +41,47 @@ export function confirmDialog({
   });
 }
 
-export class AlertService extends SNAlertService {
+export class AlertService implements SNAlertService {
-  async alert(
+  alert(
     text: string,
     title: string,
     closeButtonText = 'OK',
-    onClose: () => void
   ) {
-    return new Promise((resolve) => {
+    return new Promise<void>((resolve) => {
-      const buttons = [
+      const alert = new SKAlert({
-        {
+        title,
+        text,
+        buttons: [{
           text: closeButtonText,
           style: 'neutral',
-          action: async () => {
+          action: resolve,
-            if (onClose) {
+        }],
-              this.deviceInterface!.timeout(onClose);
+      });
-            }
-            resolve(true);
-          },
-        },
-      ];
-      const alert = new SKAlert({ title, text, buttons });
       alert.present();
     });
   }
 
-  /** @deprecated use confirmDialog instead */
+  confirm(
-  async confirm(
     text: string,
-    title: string,
+    title?: string,
-    confirmButtonText = 'Confirm',
+    confirmButtonText?: string,
-    cancelButtonText = 'Cancel',
+    confirmButtonType?: ButtonType,
-    onConfirm?: () => void,
+    cancelButtonText?: string
-    onCancel?: () => void,
+  ): Promise<boolean> {
-    destructive = false
-  ) {
     return confirmDialog({
       text,
       title,
       confirmButtonText,
       cancelButtonText,
-      confirmButtonStyle: destructive ? 'danger' : 'info'
+      confirmButtonStyle: confirmButtonType === ButtonType.Danger ? 'danger' : 'info'
-    }).then(confirmed => new Promise((resolve, reject) => {
+    });
-      if (confirmed) {
-        onConfirm?.();
-        resolve(true);
-      } else {
-        onCancel?.();
-        reject(false);
   }
-    }))
+
+  blockingDialog(text: string) {
+    const alert = new SKAlert({ text });
+    alert.present();
+    return () => {
+      alert.dismiss();
+    };
   }
 }

app/assets/javascripts/ui_models/application.ts

@@ -63,13 +63,9 @@ export class WebApplication extends SNApplication {
       platformFromString(getPlatformString()),
       deviceInterface,
       new SNWebCrypto(),
+      new AlertService(),
       namespace,
-      [
+      undefined,
-        {
-          swap: SNAlertService,
-          with: AlertService
-        }
-      ],
       undefined,
     );
     this.$compile = $compile;

app/assets/javascripts/views/editor/editor_view.ts

@@ -32,6 +32,7 @@ import {
   StringDeleteNote,
   StringEmptyTrash
 } from '@/strings';
+import { confirmDialog } from '@/services/alertService';
 
 const NOTE_PREVIEW_CHAR_LIMIT = 80;
 const MINIMUM_STATUS_DURATION = 400;
@@ -685,7 +686,7 @@ class EditorViewCtrl extends PureViewCtrl<{}, EditorState> {
       );
       return;
     }
-    const run = () => {
+    const run = async () => {
       if (this.note.locked) {
         this.application.alertService!.alert(
           STRING_DELETE_LOCKED_ATTEMPT
@@ -699,12 +700,10 @@ class EditorViewCtrl extends PureViewCtrl<{}, EditorState> {
         title,
         permanently
       );
-      this.application.alertService!.confirm(
+      if (await confirmDialog({
         text,
-        undefined,
+        confirmButtonStyle: 'danger'
-        undefined,
+      })) {
-        undefined,
-        () => {
         if (permanently) {
           this.performNoteDeletion(this.note);
         } else {
@@ -717,10 +716,7 @@ class EditorViewCtrl extends PureViewCtrl<{}, EditorState> {
             }
           );
         }
-        },
+      };
-        undefined,
-        true,
-      );
     };
     const requiresPrivilege = await this.application.privilegesService!.actionRequiresPrivilege(
       ProtectedAction.DeleteNote
@@ -761,20 +757,15 @@ class EditorViewCtrl extends PureViewCtrl<{}, EditorState> {
     return this.application.getTrashedItems().length;
   }
 
-  emptyTrash() {
+  async emptyTrash() {
     const count = this.getTrashCount();
-    this.application.alertService!.confirm(
+    if (await confirmDialog({
-      StringEmptyTrash(count),
+      text: StringEmptyTrash(count),
-      undefined,
+      confirmButtonStyle: 'danger'
-      undefined,
+    })) {
-      undefined,
-      () => {
       this.application.emptyTrash();
       this.application.sync();
-      },
+    }
-      undefined,
-      true,
-    );
   }
 
   togglePin() {