From f0b7bdef172f5f2230bea59f56708c435de6e929 Mon Sep 17 00:00:00 2001
From: Philipinho <16838612+Philipinho@users.noreply.github.com>
Date: Mon, 25 May 2026 15:55:56 +0100
Subject: [PATCH] fix(bases): give kanban its own scroll container so column
 auto-scroll works

---
 .../features/base/components/base-view.tsx    | 78 ++++++++++++++-----
 1 file changed, 57 insertions(+), 21 deletions(-)

diff --git a/apps/client/src/features/base/components/base-view.tsx b/apps/client/src/features/base/components/base-view.tsx
index 54328ae84..489c0da4c 100644
--- a/apps/client/src/features/base/components/base-view.tsx
+++ b/apps/client/src/features/base/components/base-view.tsx
@@ -190,6 +190,8 @@ export function BaseView({ pageId, embedded }: BaseViewProps) {
     baselineConfig: activeView?.config,
   });
 
+  const isKanban = effectiveView?.type === "kanban";
+
   const handleCellUpdate = useCallback(
     (rowId: string, propertyId: string, value: unknown) => {
       updateRowMutation.mutate({
@@ -389,6 +391,7 @@ export function BaseView({ pageId, embedded }: BaseViewProps) {
           base={base}
           rows={rows}
           openRowId={openRowId}
+          canEdit={canSave}
           onClose={closeRow}
         />
       </>
@@ -405,32 +408,65 @@ export function BaseView({ pageId, embedded }: BaseViewProps) {
       >
         {banner}
         {toolbar}
-        <div className={classes.tableScrollport} ref={setScrollportEl}>
-          <ViewRenderer
-            base={base}
-            rows={rows}
-            effectiveView={effectiveView}
-            table={table}
-            pageId={pageId}
-            embedded={embedded}
-            hasNextPage={!!hasNextPage}
-            isFetchingNextPage={isFetchingNextPage}
-            onFetchNextPage={fetchNextPage}
-            onCellUpdate={handleCellUpdate}
-            onAddRow={handleAddRow}
-            onColumnReorder={handleColumnReorder}
-            onResizeEnd={handleResizeEnd}
-            onRowReorder={handleRowReorder}
-            onCardClick={handleCardClick}
-            persistViewConfig={persistViewConfig}
-            scrollportEl={scrollportEl}
-          />
-        </div>
+        {isKanban ? (
+          <div
+            ref={setScrollportEl}
+            style={{
+              flex: 1,
+              minHeight: 0,
+              display: "flex",
+              flexDirection: "column",
+            }}
+          >
+            <ViewRenderer
+              base={base}
+              rows={rows}
+              effectiveView={effectiveView}
+              table={table}
+              pageId={pageId}
+              embedded={embedded}
+              hasNextPage={!!hasNextPage}
+              isFetchingNextPage={isFetchingNextPage}
+              onFetchNextPage={fetchNextPage}
+              onCellUpdate={handleCellUpdate}
+              onAddRow={handleAddRow}
+              onColumnReorder={handleColumnReorder}
+              onResizeEnd={handleResizeEnd}
+              onRowReorder={handleRowReorder}
+              onCardClick={handleCardClick}
+              persistViewConfig={persistViewConfig}
+              scrollportEl={scrollportEl}
+            />
+          </div>
+        ) : (
+          <div className={classes.tableScrollport} ref={setScrollportEl}>
+            <ViewRenderer
+              base={base}
+              rows={rows}
+              effectiveView={effectiveView}
+              table={table}
+              pageId={pageId}
+              embedded={embedded}
+              hasNextPage={!!hasNextPage}
+              isFetchingNextPage={isFetchingNextPage}
+              onFetchNextPage={fetchNextPage}
+              onCellUpdate={handleCellUpdate}
+              onAddRow={handleAddRow}
+              onColumnReorder={handleColumnReorder}
+              onResizeEnd={handleResizeEnd}
+              onRowReorder={handleRowReorder}
+              onCardClick={handleCardClick}
+              persistViewConfig={persistViewConfig}
+              scrollportEl={scrollportEl}
+            />
+          </div>
+        )}
       </div>
       <RowDetailModal
         base={base}
         rows={rows}
         openRowId={openRowId}
+        canEdit={canSave}
         onClose={closeRow}
       />
     </>