From daf1fd5e922345d5c6ffe63806f20b8914e33ef4 Mon Sep 17 00:00:00 2001
From: alma <alma@governance-labs.org>
Date: Mon, 28 Apr 2025 19:06:05 +0200
Subject: [PATCH] courrier multi account restore compose

---
 app/courrier/page.tsx | 21 ++++++++++-----------
 1 file changed, 10 insertions(+), 11 deletions(-)

diff --git a/app/courrier/page.tsx b/app/courrier/page.tsx
index caf913c9..f5b21e61 100644
--- a/app/courrier/page.tsx
+++ b/app/courrier/page.tsx
@@ -724,6 +724,16 @@ export default function CourrierPage() {
     }
   }, [selectedAccount, expandedAccounts]);
 
+  // On page load/refresh, expand all accounts so their folders are always visible
+  useEffect(() => {
+    // Expand all real accounts (not loading-account) on load/refresh
+    const expanded: Record<string, boolean> = {};
+    accounts.forEach(a => {
+      if (a.id !== 'loading-account') expanded[a.id] = true;
+    });
+    setExpandedAccounts(expanded);
+  }, [accounts]);
+
   return (
     <>
       <SimplifiedLoadingFix />
@@ -780,7 +790,6 @@ export default function CourrierPage() {
                       onClick={() => setAccountsDropdownOpen(!accountsDropdownOpen)}
                     >
                       <span>Accounts</span>
-                      {accountsDropdownOpen ? <ChevronUp className="h-4 w-4" /> : <ChevronDown className="h-4 w-4" />}
                     </Button>
                     <Button
                       variant="ghost"
@@ -1031,16 +1040,6 @@ export default function CourrierPage() {
                               </DropdownMenuContent>
                             </DropdownMenu>
                           )}
-                          {account.id !== 'loading-account' && (
-                            <button
-                              type="button"
-                              className="ml-1 text-gray-400 hover:text-gray-600 cursor-pointer flex items-center justify-center h-5 w-5"
-                              tabIndex={-1}
-                              onClick={e => { e.stopPropagation(); setExpandedAccounts(prev => ({ ...prev, [account.id]: !prev[account.id] })); }}
-                            >
-                              {expandedAccounts[account.id] ? <ChevronUp className="h-3 w-3" /> : <ChevronDown className="h-3 w-3" />}
-                            </button>
-                          )}
                         </div>
                         {/* Show folders for any expanded account */}
                         {expandedAccounts[account.id] && account.folders && account.folders.length > 0 && (