#1374: Clicking away from "Function Apps" node, and then clicking back, causes loading to never end

Author: panchagnula

AzureFunctions.AngularClient/src/app/apps-list/apps-list.component.ts

@@ -29,11 +29,19 @@ export class AppsListComponent implements OnInit, OnDestroy {
       .distinctUntilChanged()
       .switchMap(viewInfo =>{
         this.appsNode = (<AppsNode>viewInfo.node);
+        /* this is need to avoid flickering b/w no list view & table on load
+        see https://github.com/Azure/azure-functions-ux/issues/1286 */
         this.appsNode.isLoading = true;
         return (<AppsNode>viewInfo.node).childrenStream;
       })
       .subscribe(children =>{
         this.apps = children;
+
+        /* fix for https://github.com/Azure/azure-functions-ux/issues/1374 
+         if the FunctionApps node has a sibling, the below logic will need to be updated */
+        if(children.length > 0){
+          this.appsNode.isLoading = false;
+        }
         this._origRefToItems = children;
       });
 

AzureFunctions.AngularClient/src/app/tree-view/apps-node.ts

@@ -47,7 +47,6 @@ export class AppsNode extends TreeNode implements MutableCollection, Disposable,
         this.iconClass = "root-node-collection-icon"
         this.iconUrl = "images/BulletList.svg";
         this.showExpandIcon = false;
-
         this.childrenStream.subscribe(children =>{
             this.children = children;
         })
@@ -68,11 +67,10 @@ export class AppsNode extends TreeNode implements MutableCollection, Disposable,
         })
         .switchMap(result =>{
             this.childrenStream.next([]);
-
+            
             if(!result.subscriptions || result.subscriptions.length === 0){
                 return Observable.of(null);
             }
-
             this.isLoading = true;
             this._subscriptions = result.subscriptions;
             return this._doSearch(<AppNode[]>this.children, result.searchTerm, result.subscriptions, 0, null);