From eb468333c1734200d4ca92059e8b66a1fa1806c9 Mon Sep 17 00:00:00 2001
From: Amirhossein Khalili <amirhosseinkhalili901@gmail.com>
Date: Wed, 29 Apr 2026 10:42:50 +0330
Subject: [PATCH] fix(workspaces): preserve workspace thumbnail layout

---
 src/pages/Workspaces.tsx | 142 ++++++++++++++++++++-------------------
 1 file changed, 72 insertions(+), 70 deletions(-)

diff --git a/src/pages/Workspaces.tsx b/src/pages/Workspaces.tsx
index 9fb8c4e..67142b5 100644
--- a/src/pages/Workspaces.tsx
+++ b/src/pages/Workspaces.tsx
@@ -10,9 +10,9 @@ import {
   canWorkspace,
   type WorkspaceRole,
 } from '../lib/permissions';
-import FilterBar from '../components/FilterBar';
-import { ListPageSkeleton } from '../components/ListPageSkeleton';
-import { Button } from '../components/ui/button';
+import FilterBar from '../components/FilterBar';
+import { ListPageSkeleton } from '../components/ListPageSkeleton';
+import { Button } from '../components/ui/button';
 import { Input } from '../components/ui/input';
 import { Card, CardContent, CardTitle } from '../components/ui/card';
 import { Pagination } from '../components/Pagination';
@@ -116,58 +116,60 @@ export default function Workspaces() {
     }
   };
 
-  return (
-    <div className="mx-auto flex min-h-full max-w-7xl flex-col p-4 md:p-6">
-      <div className="flex flex-1 flex-col gap-5">
-      <div className="rounded-3xl border border-slate-200 bg-white p-5 shadow-sm dark:border-slate-800 dark:bg-slate-900 sm:p-6">
-        <div className="flex items-start justify-between gap-4">
-          <div>
-            <h1 className="text-2xl font-bold text-slate-900 dark:text-white">{t.workspace?.title || 'Workspaces'}</h1>
-            <p className="mt-1 text-sm text-slate-500 dark:text-slate-400">{t.workspace?.subtitle || 'Manage your workspaces'}</p>
-          </div>
-          <Button
-            onClick={() => navigate('/workspaces/create')}
-            size="icon"
-            className="shrink-0 shadow-sm"
-            title={t.workspace?.createNew || 'Create New'}
-          >
-            <Plus className="h-5 w-5" />
-          </Button>
-        </div>
-      </div>
-
-      <div className="rounded-3xl border border-slate-200 bg-white p-4 shadow-sm dark:border-slate-800 dark:bg-slate-900 sm:p-5">
-        <FilterBar 
-          searchQuery={searchQuery}
-          setSearchQuery={setSearchQuery}
-          ordering={ordering}
-          setOrdering={setOrdering}
-          orderingOptions={orderingOptions}
-          searchPlaceholder={t.workspace?.searchPlaceholder || 'Search...'}
-        />
-      </div>
-
-      {isLoading ? (
-        <ListPageSkeleton variant="list" />
-      ) : (
-        <div className="flex flex-1 flex-col gap-6">
-          <div className="flex flex-1 flex-col gap-4">
-            {workspaces.map((workspace) => {
-              const canDeleteWorkspace = canWorkspace(workspace.my_role, WORKSPACE_DELETE);
-              const canEditWorkspace = canWorkspace(workspace.my_role, WORKSPACE_EDIT);
+  return (
+    <div className="mx-auto flex min-h-full max-w-7xl flex-col p-4 md:p-6">
+      <div className="flex flex-1 flex-col gap-5">
+      <div className="rounded-3xl border border-slate-200 bg-white p-5 shadow-sm dark:border-slate-800 dark:bg-slate-900 sm:p-6">
+        <div className="flex items-start justify-between gap-4">
+          <div>
+            <h1 className="text-2xl font-bold text-slate-900 dark:text-white">{t.workspace?.title || 'Workspaces'}</h1>
+            <p className="mt-1 text-sm text-slate-500 dark:text-slate-400">{t.workspace?.subtitle || 'Manage your workspaces'}</p>
+          </div>
+          <Button
+            onClick={() => navigate('/workspaces/create')}
+            size="icon"
+            className="shrink-0 shadow-sm"
+            title={t.workspace?.createNew || 'Create New'}
+          >
+            <Plus className="h-5 w-5" />
+          </Button>
+        </div>
+      </div>
+
+      <div className="rounded-3xl border border-slate-200 bg-white p-4 shadow-sm dark:border-slate-800 dark:bg-slate-900 sm:p-5">
+        <FilterBar 
+          searchQuery={searchQuery}
+          setSearchQuery={setSearchQuery}
+          ordering={ordering}
+          setOrdering={setOrdering}
+          orderingOptions={orderingOptions}
+          searchPlaceholder={t.workspace?.searchPlaceholder || 'Search...'}
+        />
+      </div>
+
+      {isLoading ? (
+        <ListPageSkeleton variant="list" />
+      ) : (
+        <div className="flex flex-1 flex-col gap-6">
+          <div className="flex flex-1 flex-col gap-4">
+            {workspaces.map((workspace) => {
+              const canDeleteWorkspace = canWorkspace(workspace.my_role, WORKSPACE_DELETE);
+              const canEditWorkspace = canWorkspace(workspace.my_role, WORKSPACE_EDIT);
 
               return (
                 <Card key={workspace.id} className="flex flex-col text-slate-800 dark:text-slate-100 dark:bg-slate-800 dark:border-slate-700 shadow-sm">
                   <CardContent className="flex flex-col sm:flex-row items-start sm:items-center justify-between py-4 px-6 gap-4">
                     <div className="flex-1">
                       <div className="flex items-center gap-3 mb-2">
-                        <div className="h-9 w-9 shrink-0 overflow-hidden rounded-lg bg-slate-100 dark:bg-slate-600 flex items-center justify-center text-sm font-semibold text-slate-700 dark:text-slate-200">
-                          {workspace.thumbnail ? (
+                        {workspace.thumbnail ? (
+                          <div className="h-9 w-9 shrink-0 overflow-hidden rounded-lg flex items-center justify-center text-sm font-semibold text-slate-700 dark:text-slate-200">
                             <img src={workspace.thumbnail} alt={workspace.name} className="h-full w-full object-cover" />
-                          ) : (
-                            workspace.name.trim().charAt(0).toUpperCase() || "W"
-                          )}
-                        </div>
+                          </div>
+                        ) : (
+                          <div className="h-9 w-9 shrink-0 overflow-hidden rounded-lg bg-slate-100 dark:bg-slate-600 flex items-center justify-center text-sm font-semibold text-slate-700 dark:text-slate-200">
+                            {workspace.name.trim().charAt(0).toUpperCase() || "W"}
+                          </div>
+                        )}
                         <CardTitle className="text-lg line-clamp-1">
                           {workspace.name}
                         </CardTitle>
@@ -215,31 +217,31 @@ export default function Workspaces() {
                   </CardContent>
                 </Card>
               );
-            })}
-
-            {workspaces.length === 0 && (
-              <div className="flex flex-1 rounded-3xl border-2 border-dashed border-slate-200 bg-white py-16 shadow-sm dark:border-slate-800 dark:bg-slate-900">
-                <div className="flex flex-col items-center justify-center">
-                <p className="text-slate-500 dark:text-slate-400 font-medium">{t.workspace?.emptyState || 'No workspaces found'}</p>
-                </div>
-              </div>
-            )}
-          </div>
-
-          <Pagination
+            })}
+
+            {workspaces.length === 0 && (
+              <div className="flex flex-1 rounded-3xl border-2 border-dashed border-slate-200 bg-white py-16 shadow-sm dark:border-slate-800 dark:bg-slate-900">
+                <div className="flex flex-col items-center justify-center">
+                <p className="text-slate-500 dark:text-slate-400 font-medium">{t.workspace?.emptyState || 'No workspaces found'}</p>
+                </div>
+              </div>
+            )}
+          </div>
+
+          <Pagination
               currentPage={currentPage}
               totalCount={totalItems}
               limit={limit}
-              onPageChange={setCurrentPage}
-              onLimitChange={setLimit}
-              pageSizeOptions={[10, 20, 50]}
-            />
-        </div>
-      )}
-      </div>
-
-      {deleteModal.workspace && (
-        <Modal
+              onPageChange={setCurrentPage}
+              onLimitChange={setLimit}
+              pageSizeOptions={[10, 20, 50]}
+            />
+        </div>
+      )}
+      </div>
+
+      {deleteModal.workspace && (
+        <Modal
           isOpen={deleteModal.isOpen}
           onClose={() => {
             setDeleteModal({ isOpen: false, workspace: null });