Skip to content

Commit

Permalink
COL-1069, no 'No assets' msg until NOT isLoading
Browse files Browse the repository at this point in the history
  • Loading branch information
johncrossman committed Jun 28, 2017
1 parent 5975936 commit 512dff2
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 1 deletion.
4 changes: 4 additions & 0 deletions public/app/assetlibrary/list/assetLibraryListController.js
Original file line number Diff line number Diff line change
Expand Up @@ -46,6 +46,9 @@
'ready': true
};

// False once getAssets is done
$scope.isLoading = true;

// Variable that will keep track of the scroll position in the list
var scrollPosition = 0;

Expand Down Expand Up @@ -121,6 +124,7 @@
$scope.resultsMessage = utilService.buildSearchResultsMessage('Displaying assets', $scope.searchOptions);
}
}
$scope.isLoading = false;
});
// Ensure that the next page is requested the next time
$scope.list.page++;
Expand Down
2 changes: 1 addition & 1 deletion public/app/assetlibrary/list/list.html
Original file line number Diff line number Diff line change
Expand Up @@ -70,7 +70,7 @@
</li>
</ul>
</div>
<div class="alert alert-info assetlibrary-list-noresults" data-ng-if="isSearch && assets.length === 0">
<div class="alert alert-info assetlibrary-list-noresults" data-ng-if="!isLoading && isSearch && assets.length === 0">
No matching assets were found.
</div>
<!-- COPYRIGHT -->
Expand Down

0 comments on commit 512dff2

Please sign in to comment.