fix: inactive view reloads document for active view

Author: ycnmhd

src/lang/lang.ts

@@ -180,7 +180,7 @@ export const lang = {
     hk_extend_select_to_end_of_group: 'Extend selection to end of group',
     hk_extend_select_to_start_of_column: 'Extend selection to start of column',
     hk_extend_select_to_end_of_column: 'Extend selection to end of column',
-    hkg_selection: 'Selection',
+    hkg_selection: 'Select',
     hk_select_all: 'Select all sections',
 
     // navigate spatially
@@ -195,7 +195,7 @@ export const lang = {
     hk_go_to_beginning_of_column: 'Go to start of column',
     hk_go_to_end_of_column: 'Go to end of column',
     hk_select_parent: 'Select parent section',
-    hkg_navigation: 'Navigation',
+    hkg_navigation: 'Navigate',
     hk_select_previous_sibling: 'Select previous sibling',
     hk_select_next_sibling: 'Select next sibling',
 

src/stores/view/subscriptions/on-document-state-update.ts

@@ -84,8 +84,7 @@ export const onDocumentStateUpdate = (
     if (!container || !view.isViewOfFile) return;
 
     if (e.content || structuralChange) {
-        const debouncedSave = type === 'DOCUMENT/MOVE_NODE';
-        view.saveDocument(debouncedSave);
+        view.saveDocument();
     }
 
     if (e.content || structuralChange) {

src/view/view.ts

@@ -70,7 +70,6 @@ export class LineageView extends TextFileView {
     id: string;
     zoomFactor: number;
     minimapDom: MinimapDomElements | null = null;
-    private hasDebouncedSave = false;
     private readonly onDestroyCallbacks: Set<Unsubscriber> = new Set();
     private activeFilePath: null | string;
     constructor(
@@ -121,16 +120,11 @@ export class LineageView extends TextFileView {
             this.loadInitialData();
         } else {
             this.data = data;
-            this.debouncedLoadDocumentToStore();
+            if (this.isViewOfFile) this.debouncedLoadDocumentToStore();
         }
     }
 
     async onUnloadFile() {
-        if (this.hasDebouncedSave && this.data.length) {
-            this.save();
-            this.hasDebouncedSave = false;
-        }
-
         if (this.component) {
             this.component.$destroy();
         }
@@ -191,7 +185,7 @@ export class LineageView extends TextFileView {
         onPluginError(error, location, action);
     };
 
-    saveDocument = async (debounced = false) => {
+    saveDocument = async () => {
         invariant(this.file);
         const state = clone(this.documentStore.getValue());
         const data: string =
@@ -202,25 +196,10 @@ export class LineageView extends TextFileView {
                 throw new Error(lang.error_save_empty_data);
             }
             this.data = data;
-            if (debounced) {
-                this.debouncedSave();
-            } else {
-                this.requestSave();
-            }
-            this.hasDebouncedSave = debounced;
+            this.requestSave();
         }
     };
 
-    private debouncedSave = debounce(
-        () => {
-            if (!this.hasDebouncedSave) return;
-            this.save();
-            this.hasDebouncedSave = false;
-        },
-        8000,
-        true,
-    );
-
     private loadInitialData = async () => {
         invariant(this.file);