Merge pull request #92 from DDMAL/integrate-changes

Set to unsaved after edit actions other than cell editing

src/Editor/CressTable.ts

@@ -6,6 +6,20 @@ import { ColumnTools } from './ColumnTools';
 import { updateAttachment } from '../Dashboard/Storage';
 import { setSavedStatus } from '../utils/Unsaved';
 import * as Notification from '../utils/Notification';
+import { TableEvent } from '../Types';
+
+const changeHooks: TableEvent[] = [
+  'afterChange',
+  'afterColumnMove',
+  'afterColumnSequenceChange',
+  'afterCreateCol',
+  'afterCreateRow',
+  'afterCut',
+  'afterRemoveCol',
+  'afterRemoveRow',
+  'afterRowMove',
+  'afterRowSequenceChange',
+];
 
 export class CressTable {
   private table: Handsontable;
@@ -65,15 +79,15 @@ export class CressTable {
       dropdownMenu: true,
       className: 'table-menu-btn',
       licenseKey: 'non-commercial-and-evaluation',
-      afterChange(changes, source) {
-        if (source != 'loadData') setSavedStatus(false);
+      afterChange() {
         this.validateCells();
       },
       beforeValidate: (value) => this.setProcessStatus(value),
       afterValidate: (isValid) => this.setResultStatus(isValid),
     });
 
     this.initFileListener(id, inputHeader, body, headers);
+    this.initChangeListener();
   }
 
   private initFileListener(
@@ -141,4 +155,12 @@ export class CressTable {
       this.ColumnTools.hasInvalid = false;
     }
   }
+
+  private initChangeListener() {
+    changeHooks.forEach((hook) => {
+      this.table.addHook(hook, (source) => {
+        if (source != 'loadData') setSavedStatus(false);
+      });
+    });
+  }
 }

src/Types.ts

@@ -1,4 +1,5 @@
 import PouchDB from 'pouchdb';
+
 interface Glyph {
   imagePath: string | null;
   imageBinary: string | null;
@@ -61,3 +62,15 @@ export type validationStatus = 'unknown' | 'processing' | 'done';
 export type HTMLSVGElement = HTMLElement & SVGSVGElement;
 
 export type NotificationType = 'default' | 'error' | 'warning' | 'success';
+
+export type TableEvent =
+  | 'afterChange'
+  | 'afterColumnMove'
+  | 'afterColumnSequenceChange'
+  | 'afterCreateCol'
+  | 'afterCreateRow'
+  | 'afterCut'
+  | 'afterRemoveCol'
+  | 'afterRemoveRow'
+  | 'afterRowMove'
+  | 'afterRowSequenceChange';