From 529ab2948934a315e6310b5b4aef2158d068f097 Mon Sep 17 00:00:00 2001
From: Qstick <qstick@gmail.com>
Date: Sun, 27 Jan 2019 00:22:24 -0500
Subject: [PATCH] Fixed: Show Loading Indicator for Manage TrackFiles Modal

---
 .../TrackFile/Editor/TrackFileEditorModalContent.js  | 12 ++++++++++--
 .../Editor/TrackFileEditorModalContentConnector.js   |  1 +
 2 files changed, 11 insertions(+), 2 deletions(-)

diff --git a/frontend/src/TrackFile/Editor/TrackFileEditorModalContent.js b/frontend/src/TrackFile/Editor/TrackFileEditorModalContent.js
index 6c480374a..e3caf692d 100644
--- a/frontend/src/TrackFile/Editor/TrackFileEditorModalContent.js
+++ b/frontend/src/TrackFile/Editor/TrackFileEditorModalContent.js
@@ -7,6 +7,7 @@ import toggleSelected from 'Utilities/Table/toggleSelected';
 import { kinds } from 'Helpers/Props';
 import ConfirmModal from 'Components/Modal/ConfirmModal';
 import Button from 'Components/Link/Button';
+import LoadingIndicator from 'Components/Loading/LoadingIndicator';
 import SpinnerButton from 'Components/Link/SpinnerButton';
 import SelectInput from 'Components/Form/SelectInput';
 import ModalContent from 'Components/Modal/ModalContent';
@@ -133,6 +134,7 @@ class TrackFileEditorModalContent extends Component {
   render() {
     const {
       isDeleting,
+      isFetching,
       items,
       languages,
       qualities,
@@ -174,14 +176,19 @@ class TrackFileEditorModalContent extends Component {
 
         <ModalBody>
           {
-            !items.length &&
+            isFetching &&
+              <LoadingIndicator />
+          }
+
+          {
+            !isFetching && !items.length &&
               <div>
                 No track files to manage.
               </div>
           }
 
           {
-            !!items.length &&
+            !isFetching && !!items.length &&
               <Table
                 columns={columns}
                 selectAll={true}
@@ -262,6 +269,7 @@ class TrackFileEditorModalContent extends Component {
 
 TrackFileEditorModalContent.propTypes = {
   isDeleting: PropTypes.bool.isRequired,
+  isFetching: PropTypes.bool.isRequired,
   items: PropTypes.arrayOf(PropTypes.object).isRequired,
   languages: PropTypes.arrayOf(PropTypes.object).isRequired,
   qualities: PropTypes.arrayOf(PropTypes.object).isRequired,
diff --git a/frontend/src/TrackFile/Editor/TrackFileEditorModalContentConnector.js b/frontend/src/TrackFile/Editor/TrackFileEditorModalContentConnector.js
index 91aedfb71..66e8e92d6 100644
--- a/frontend/src/TrackFile/Editor/TrackFileEditorModalContentConnector.js
+++ b/frontend/src/TrackFile/Editor/TrackFileEditorModalContentConnector.js
@@ -59,6 +59,7 @@ function createMapStateToProps() {
         items,
         artistType: artist.artistType,
         isDeleting: trackFiles.isDeleting,
+        isFetching: tracks.isFetching || trackFiles.isFetching,
         isSaving: trackFiles.isSaving,
         languages,
         qualities