[bugfix] Task Component Fixes (#84)

* Fixed Ghost Task showing during any task loading

* Fixed issue with Todo DataSource reseting target drawer

Author: benotter

src/components/mgt-tasks/mgt-tasks.ts

@@ -93,6 +93,8 @@ export class MgtTasks extends MgtBaseComponent {
   @property() private _loadingTasks: string[] = [];
 
   @property() private _inTaskLoad: boolean = false;
+  @property() private _hasDoneInitialLoad: boolean = false;
+  @property() private _todoDefaultSet: boolean = false;
 
   private _me: User = null;
   private _providerUpdateCallback: () => void | any;
@@ -154,6 +156,10 @@ export class MgtTasks extends MgtBaseComponent {
       this._drawers = [];
       this._dressers = [];
 
+      this._hasDoneInitialLoad = false;
+      this._inTaskLoad = false;
+      this._todoDefaultSet = false;
+
       this.loadTasks();
     }
   }
@@ -177,6 +183,8 @@ export class MgtTasks extends MgtBaseComponent {
     }
 
     this._inTaskLoad = false;
+
+    if (!this._hasDoneInitialLoad) this._hasDoneInitialLoad = true;
   }
 
   private async _loadTargetTodoTasks(ts: ITaskSource) {
@@ -221,7 +229,8 @@ export class MgtTasks extends MgtBaseComponent {
       []
     );
 
-    if (!this.initialId) {
+    if (!this.initialId && this.dataSource === 'todo' && !this._todoDefaultSet) {
+      this._todoDefaultSet = true;
       let defaultDrawer = drawers.find(d => (d._raw as OutlookTaskFolder).isDefaultFolder);
       if (defaultDrawer) this._currentTargetDrawer = defaultDrawer.id;
     }
@@ -310,7 +319,7 @@ export class MgtTasks extends MgtBaseComponent {
       .filter(task => this.taskBucketPlanFilter(task))
       .filter(task => !this._hiddenTasks.includes(task.id));
 
-    let loadingTask = this._inTaskLoad ? this.renderLoadingTask() : null;
+    let loadingTask = this._inTaskLoad && !this._hasDoneInitialLoad ? this.renderLoadingTask() : null;
 
     return html`
       <div class="Header">
@@ -352,7 +361,7 @@ export class MgtTasks extends MgtBaseComponent {
 
     if (!p || p.state !== ProviderState.SignedIn) return null;
 
-    if (this._inTaskLoad)
+    if (this._inTaskLoad && !this._hasDoneInitialLoad)
       return html`
         <span class="LoadingHeader"></span>
       `;