From 2da5eaa2a5afbd85b959f8a6d094abb2ddf2f0a0 Mon Sep 17 00:00:00 2001
From: Connor Yoh <connor@stirlingpdf.com>
Date: Thu, 14 Aug 2025 15:34:55 +0100
Subject: [PATCH] Files collapsed state fix

---
 frontend/src/tools/AddPassword.tsx       | 11 ++++-------
 frontend/src/tools/ChangePermissions.tsx |  2 +-
 2 files changed, 5 insertions(+), 8 deletions(-)

diff --git a/frontend/src/tools/AddPassword.tsx b/frontend/src/tools/AddPassword.tsx
index 667252680..55b0f0ea8 100644
--- a/frontend/src/tools/AddPassword.tsx
+++ b/frontend/src/tools/AddPassword.tsx
@@ -63,19 +63,17 @@ const AddPassword = ({ onPreviewFile, onComplete, onError }: BaseToolProps) => {
 
   const hasFiles = selectedFiles.length > 0;
   const hasResults = addPasswordOperation.files.length > 0 || addPasswordOperation.downloadUrl !== null;
-  const passwordsCollapsed = !hasFiles || hasResults;
-  const permissionsCollapsed = collapsedPermissions || hasResults;
+  const optionsCollapsed = !hasFiles || hasResults;
 
   return createToolFlow({
     files: {
       selectedFiles,
-      isCollapsed: hasFiles && !hasResults,
+      isCollapsed: hasFiles || hasResults,
     },
     steps: [
       {
         title: t('addPassword.passwords.stepTitle', 'Passwords & Encryption'),
-        isVisible: hasFiles,
-        isCollapsed: passwordsCollapsed,
+        isCollapsed: optionsCollapsed,
         onCollapsedClick: hasResults ? handleSettingsReset : undefined,
         tooltip: addPasswordTips,
         content: (
@@ -88,8 +86,7 @@ const AddPassword = ({ onPreviewFile, onComplete, onError }: BaseToolProps) => {
       },
       {
         title: t('addPassword.permissions.stepTitle', 'Document Permissions'),
-        isVisible: hasFiles,
-        isCollapsed: permissionsCollapsed,
+        isCollapsed: optionsCollapsed,
         onCollapsedClick: hasResults ? handleSettingsReset : () => setCollapsedPermissions(!collapsedPermissions),
         content: (
           <ChangePermissionsSettings
diff --git a/frontend/src/tools/ChangePermissions.tsx b/frontend/src/tools/ChangePermissions.tsx
index 19e3d859f..f92cbab73 100644
--- a/frontend/src/tools/ChangePermissions.tsx
+++ b/frontend/src/tools/ChangePermissions.tsx
@@ -65,7 +65,7 @@ const ChangePermissions = ({ onPreviewFile, onComplete, onError }: BaseToolProps
   return createToolFlow({
     files: {
       selectedFiles,
-      isCollapsed: hasFiles && !hasResults,
+      isCollapsed: hasFiles || hasResults,
     },
     steps: [{
       title: t('settings.title', 'Settings'),