fix: wait for CompletedFullSync before showing "No Notes."

standardnotes · Sep 23, 2020 · 5c98633 · 5c98633
1 parent f819513
commit 5c98633
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 6 deletions.
diff --git a/app/assets/javascripts/views/notes/notes-view.pug b/app/assets/javascripts/views/notes/notes-view.pug
@@ -95,8 +95,11 @@
                   label="'Date'"
                   )
     p.empty-notes-list.faded(
-      ng-if="self.state.localDataLoaded && !self.state.renderedNotes.length"
+      ng-if="self.state.completedFullSync && !self.state.renderedNotes.length"
     ) No notes.
+    p.empty-notes-list.faded(
+      ng-if="!self.state.completedFullSync && !self.state.renderedNotes.length"
+    ) Loading notes…
     .scrollable(ng-if="self.state.renderedNotes.length")
       #notes-scrollable.infinite-scroll(
         can-load='true',

diff --git a/app/assets/javascripts/views/notes/notes_view.ts b/app/assets/javascripts/views/notes/notes_view.ts
@@ -34,7 +34,7 @@ type NotesState = {
   hideDate?: boolean
   noteFilter: { text: string }
   mutable: { showMenu: boolean }
-  localDataLoaded: boolean
+  completedFullSync: boolean
   [WebPrefKey.TagsPanelWidth]?: number
   [WebPrefKey.NotesPanelWidth]?: number
   [WebPrefKey.EditorWidth]?: number
@@ -127,7 +127,7 @@ class NotesViewCtrl extends PureViewCtrl<{}, NotesState> {
       mutable: { showMenu: false },
       noteFilter: { text: '' },
       panelTitle: '',
-      localDataLoaded: false,
+      completedFullSync: false,
     };
   }
 
@@ -167,17 +167,18 @@ class NotesViewCtrl extends PureViewCtrl<{}, NotesState> {
       case ApplicationEvent.SignedIn:
         this.appState.closeAllEditors();
         this.selectFirstNote();
+        this.setState({
+          completedFullSync: false,
+        });
         break;
       case ApplicationEvent.CompletedFullSync:
         this.getMostValidNotes().then((notes) => {
           if (notes.length === 0) {
             this.createPlaceholderNote();
           }
         });
-        break;
-      case ApplicationEvent.LocalDataLoaded:
         this.setState({
-          localDataLoaded: true,
+          completedFullSync: true,
         });
         break;
     }