fix: tenants/tablets scroll

astandrik · astandrik · commit e058b5408490 · 2025-05-20T22:04:22.000+03:00
diff --git a/src/containers/Cluster/Cluster.scss b/src/containers/Cluster/Cluster.scss
@@ -63,8 +63,6 @@
     }
 
     .ydb-table-with-controls-layout {
-        // Height of visible tabs area above table for min-height calculation
-        --data-table-min-height-offset: 62px;
         // Total height of all fixed elements above table for sticky header positioning
         --data-table-sticky-header-offset: 102px;
     }
diff --git a/src/containers/Tablets/TabletsTable.tsx b/src/containers/Tablets/TabletsTable.tsx
@@ -214,7 +214,7 @@ export function TabletsTable({
             {error ? <ResponseError error={error} /> : null}
             <TableWithControlsLayout.Table
                 scrollContainerRef={scrollContainerRef}
-                scrollDependencies={[filteredTablets]}
+                scrollDependencies={[tabletsSearch]}
                 loading={loading}
             >
                 <ResizeableDataTable
diff --git a/src/containers/Tenants/Tenants.tsx b/src/containers/Tenants/Tenants.tsx
@@ -311,7 +311,7 @@ export const Tenants = ({additionalTenantsProps, scrollContainerRef}: TenantsPro
                 <TableWithControlsLayout.Table
                     scrollContainerRef={scrollContainerRef}
                     loading={loading}
-                    scrollDependencies={[filteredTenants]}
+                    scrollDependencies={[searchValue, problemFilter]}
                 >
                     {currentData ? renderTable() : null}
                 </TableWithControlsLayout.Table>

Original file line number	Diff line number	Diff line change
`@@ -63,8 +63,6 @@`
`63`	`63`	`}`
`64`	`64`
`65`	`65`	`.ydb-table-with-controls-layout {`
`66`		`- // Height of visible tabs area above table for min-height calculation`
`67`		`- --data-table-min-height-offset: 62px;`
`68`	`66`	`// Total height of all fixed elements above table for sticky header positioning`
`69`	`67`	`--data-table-sticky-header-offset: 102px;`
`70`	`68`	`}`
Original file line number	Diff line number	Diff line change
`@@ -214,7 +214,7 @@ export function TabletsTable({`
`214`	`214`	`{error ? <ResponseError error={error} /> : null}`
`215`	`215`	`<TableWithControlsLayout.Table`
`216`	`216`	`scrollContainerRef={scrollContainerRef}`
`217`		`- scrollDependencies={[filteredTablets]}`
	`217`	`+ scrollDependencies={[tabletsSearch]}`
`218`	`218`	`loading={loading}`
`219`	`219`	`>`
`220`	`220`	`<ResizeableDataTable`
Original file line number	Diff line number	Diff line change
`@@ -311,7 +311,7 @@ export const Tenants = ({additionalTenantsProps, scrollContainerRef}: TenantsPro`
`311`	`311`	`<TableWithControlsLayout.Table`
`312`	`312`	`scrollContainerRef={scrollContainerRef}`
`313`	`313`	`loading={loading}`
`314`		`- scrollDependencies={[filteredTenants]}`
	`314`	`+ scrollDependencies={[searchValue, problemFilter]}`
`315`	`315`	`>`
`316`	`316`	`{currentData ? renderTable() : null}`
`317`	`317`	`</TableWithControlsLayout.Table>`