From 6f3ff0c85db35983b2b255dbc50383724d9a1a87 Mon Sep 17 00:00:00 2001
From: Madeline Lumetta <mlumetta@loc.gov>
Date: Fri, 1 Sep 2023 15:44:06 -0400
Subject: [PATCH] Hide upload button when state is recovered if
 hideUploadButton is true

---
 packages/@uppy/status-bar/src/StatusBarUI.jsx | 83 ++++++++++---------
 1 file changed, 46 insertions(+), 37 deletions(-)

diff --git a/packages/@uppy/status-bar/src/StatusBarUI.jsx b/packages/@uppy/status-bar/src/StatusBarUI.jsx
index a24924d492..67f19b7789 100644
--- a/packages/@uppy/status-bar/src/StatusBarUI.jsx
+++ b/packages/@uppy/status-bar/src/StatusBarUI.jsx
@@ -122,12 +122,11 @@ export default function StatusBar (props) {
 
   const width = progressValue ?? 100
 
-  const showUploadBtn = !error
-    && newFiles
-    && !isUploadInProgress
-    && !isAllPaused
-    && allowNewUpload
-    && !hideUploadButton
+  const showUploadBtn = !hideUploadButton
+    && (
+      (!error && newFiles && !isUploadInProgress && !isAllPaused && allowNewUpload)
+      || recoveredState
+    )
 
   const showCancelBtn = !hideCancelButton
     && uploadState !== STATE_WAITING
@@ -141,6 +140,14 @@ export default function StatusBar (props) {
 
   const showDoneBtn = doneButtonHandler && uploadState === STATE_COMPLETE
 
+  const hasActions = (
+    showUploadBtn
+    || showCancelBtn
+    || showPauseResumeBtn
+    || showRetryBtn
+    || showDoneBtn
+  )
+
   const progressClassNames = classNames('uppy-StatusBar-progress', {
     'is-indeterminate': getIsIndeterminate(),
   })
@@ -204,37 +211,39 @@ export default function StatusBar (props) {
         }
       })()}
 
-      <div className="uppy-StatusBar-actions">
-        {recoveredState || showUploadBtn ? (
-          <UploadBtn
-            newFiles={newFiles}
-            isUploadStarted={isUploadStarted}
-            recoveredState={recoveredState}
-            i18n={i18n}
-            isSomeGhost={isSomeGhost}
-            startUpload={startUpload}
-            uploadState={uploadState}
-          />
-        ) : null}
-
-        {showRetryBtn ? <RetryBtn i18n={i18n} uppy={uppy} /> : null}
-
-        {showPauseResumeBtn ? (
-          <PauseResumeButton
-            isAllPaused={isAllPaused}
-            i18n={i18n}
-            isAllComplete={isAllComplete}
-            resumableUploads={resumableUploads}
-            uppy={uppy}
-          />
-        ) : null}
-
-        {showCancelBtn ? <CancelBtn i18n={i18n} uppy={uppy} /> : null}
-
-        {showDoneBtn ? (
-          <DoneBtn i18n={i18n} doneButtonHandler={doneButtonHandler} />
-        ) : null}
-      </div>
+      {hasActions ? (
+        <div className="uppy-StatusBar-actions">
+          {showUploadBtn ? (
+            <UploadBtn
+              newFiles={newFiles}
+              isUploadStarted={isUploadStarted}
+              recoveredState={recoveredState}
+              i18n={i18n}
+              isSomeGhost={isSomeGhost}
+              startUpload={startUpload}
+              uploadState={uploadState}
+            />
+          ) : null}
+
+          {showRetryBtn ? <RetryBtn i18n={i18n} uppy={uppy} /> : null}
+
+          {showPauseResumeBtn ? (
+            <PauseResumeButton
+              isAllPaused={isAllPaused}
+              i18n={i18n}
+              isAllComplete={isAllComplete}
+              resumableUploads={resumableUploads}
+              uppy={uppy}
+            />
+          ) : null}
+
+          {showCancelBtn ? <CancelBtn i18n={i18n} uppy={uppy} /> : null}
+
+          {showDoneBtn ? (
+            <DoneBtn i18n={i18n} doneButtonHandler={doneButtonHandler} />
+          ) : null}
+        </div>
+      ) : null}
     </div>
   )
 }