[DataGridPro] Keep bottom pinned row at the bottom (mui#13313)

Author: romgrk

packages/x-data-grid-premium/src/tests/clipboard.DataGridPremium.test.tsx

@@ -119,12 +119,14 @@ describe('<DataGridPremium /> - Clipboard', () => {
         { id: 2, brand: 'Puma' },
       ];
       render(
-        <DataGridPremium
-          columns={columns}
-          rows={rows}
-          cellSelection
-          sortModel={[{ field: 'brand', sort: 'asc' }]}
-        />,
+        <div style={{ width: 300, height: 300 }}>
+          <DataGridPremium
+            columns={columns}
+            rows={rows}
+            cellSelection
+            sortModel={[{ field: 'brand', sort: 'asc' }]}
+          />
+        </div>,
       );
 
       const cell = getCell(0, 0);
@@ -143,15 +145,17 @@ describe('<DataGridPremium /> - Clipboard', () => {
 
     it('should not escape double quotes when copying multiple cells to clipboard', () => {
       render(
-        <DataGridPremium
-          columns={[{ field: 'value' }]}
-          rows={[
-            { id: 0, value: '1 " 1' },
-            { id: 1, value: '2' },
-          ]}
-          cellSelection
-          disableRowSelectionOnClick
-        />,
+        <div style={{ width: 300, height: 300 }}>
+          <DataGridPremium
+            columns={[{ field: 'value' }]}
+            rows={[
+              { id: 0, value: '1 " 1' },
+              { id: 1, value: '2' },
+            ]}
+            cellSelection
+            disableRowSelectionOnClick
+          />
+        </div>,
       );
 
       const cell = getCell(0, 0);

packages/x-data-grid-pro/src/tests/pagination.DataGridPro.test.tsx

@@ -105,7 +105,11 @@ describe('<DataGridPro /> - Pagination', () => {
 
   it('should log an error if rowCount is used with client-side pagination', () => {
     expect(() => {
-      render(<DataGridPro rows={[]} columns={[]} paginationMode="client" rowCount={100} />);
+      render(
+        <div style={{ width: 300, height: 300 }}>
+          <DataGridPro rows={[]} columns={[]} paginationMode="client" rowCount={100} />
+        </div>,
+      );
     }).toErrorDev([
       'MUI X: Usage of the `rowCount` prop with client side pagination (`paginationMode="client"`) has no effect. `rowCount` is only meant to be used with `paginationMode="server"`.',
     ]);

packages/x-data-grid/src/components/base/GridOverlays.tsx

@@ -69,7 +69,8 @@ function GridOverlayWrapper(props: React.PropsWithChildren<GridOverlaysProps>) {
 
   let height: React.CSSProperties['height'] =
     dimensions.viewportOuterSize.height -
-    dimensions.headersTotalHeight -
+    dimensions.topContainerHeight -
+    dimensions.bottomContainerHeight -
     (dimensions.hasScrollX ? dimensions.scrollbarSize : 0);
 
   if ((rootProps.autoHeight && currentPage.rows.length === 0) || height === 0) {

packages/x-data-grid/src/components/virtualization/GridVirtualScroller.tsx

@@ -106,7 +106,7 @@ function GridVirtualScroller(props: GridVirtualScrollerProps) {
           </RenderZone>
         </Content>
 
-        {rows.length > 0 && <SpaceFiller />}
+        <SpaceFiller rowsLength={rows.length} />
 
         <BottomContainer>
           <rootProps.slots.pinnedRows position="bottom" virtualScroller={virtualScroller} />

packages/x-data-grid/src/components/virtualization/GridVirtualScrollerFiller.tsx

@@ -17,24 +17,29 @@ const Pinned = styled('div')({
   position: 'sticky',
   height: '100%',
   boxSizing: 'border-box',
-  borderTop: '1px solid var(--DataGrid-rowBorderColor)',
+  borderTop: '1px solid var(--rowBorderColor)',
   backgroundColor: 'var(--DataGrid-pinnedBackground)',
 });
 const PinnedLeft = styled(Pinned)({
   left: 0,
-  borderRight: '1px solid var(--DataGrid-rowBorderColor)',
+  borderRight: '1px solid var(--rowBorderColor)',
 });
 const PinnedRight = styled(Pinned)({
   right: 0,
-  borderLeft: '1px solid var(--DataGrid-rowBorderColor)',
+  borderLeft: '1px solid var(--rowBorderColor)',
 });
 
 const Main = styled('div')({
   flexGrow: 1,
-  borderTop: '1px solid var(--DataGrid-rowBorderColor)',
+  borderTop: '1px solid var(--rowBorderColor)',
 });
 
-function GridVirtualScrollerFiller() {
+type Props = {
+  /** The number of rows */
+  rowsLength: number;
+};
+
+function GridVirtualScrollerFiller({ rowsLength }: Props) {
   const apiRef = useGridApiContext();
   const {
     viewportOuterSize,
@@ -54,7 +59,16 @@ function GridVirtualScrollerFiller() {
   }
 
   return (
-    <Filler className={gridClasses.filler} role="presentation" style={{ height }}>
+    <Filler
+      className={gridClasses.filler}
+      role="presentation"
+      style={
+        {
+          height,
+          '--rowBorderColor': rowsLength === 0 ? 'transparent' : 'var(--DataGrid-rowBorderColor)',
+        } as React.CSSProperties
+      }
+    >
       {leftPinnedWidth > 0 && (
         <PinnedLeft
           className={gridClasses['filler--pinnedLeft']}

packages/x-data-grid/src/hooks/features/virtualization/useGridVirtualScroller.tsx

@@ -517,22 +517,13 @@ export const useGridVirtualScroller = () => {
   );
 
   const contentSize = React.useMemo(() => {
-    // In cases where the columns exceed the available width,
-    // the horizontal scrollbar should be shown even when there're no rows.
-    // Keeping 1px as minimum height ensures that the scrollbar will visible if necessary.
-    const height = Math.max(contentHeight, 1);
-
     const size: React.CSSProperties = {
       width: needsHorizontalScrollbar ? columnsTotalWidth : 'auto',
-      height,
+      height: contentHeight,
     };
 
-    if (rootProps.autoHeight) {
-      if (currentPage.rows.length === 0) {
-        size.height = getMinimalContentHeight(apiRef); // Give room to show the overlay when there no rows.
-      } else {
-        size.height = contentHeight;
-      }
+    if (rootProps.autoHeight && currentPage.rows.length === 0) {
+      size.height = getMinimalContentHeight(apiRef); // Give room to show the overlay when there no rows.
     }
 
     return size;

packages/x-data-grid/src/tests/DataGrid.test.tsx

@@ -76,19 +76,21 @@ describe('<DataGrid />', () => {
     ];
     expect(() => {
       render(
-        <DataGrid
-          {...(
-            Object.keys(DATA_GRID_PROPS_DEFAULT_VALUES) as Array<
-              keyof typeof DATA_GRID_PROPS_DEFAULT_VALUES
-            >
-          ).reduce((acc, key) => {
-            // @ts-ignore
-            acc[key] = undefined;
-            return acc;
-          }, {})}
-          rows={rows}
-          columns={columns}
-        />,
+        <div style={{ height: 300, width: 300 }}>
+          <DataGrid
+            {...(
+              Object.keys(DATA_GRID_PROPS_DEFAULT_VALUES) as Array<
+                keyof typeof DATA_GRID_PROPS_DEFAULT_VALUES
+              >
+            ).reduce((acc, key) => {
+              // @ts-ignore
+              acc[key] = undefined;
+              return acc;
+            }, {})}
+            rows={rows}
+            columns={columns}
+          />
+        </div>,
       );
     }).not.toErrorDev();
   });

packages/x-data-grid/src/tests/layout.DataGrid.test.tsx

@@ -1174,7 +1174,7 @@ describe('<DataGrid /> - Layout & warnings', () => {
     }
 
     render(
-      <div style={{ height: '100%', width: 400.6 }}>
+      <div style={{ height: 300, width: 400.6 }}>
         <DataGrid rows={[{ id: 1 }]} columns={[{ field: 'id', flex: 1 }]} />
       </div>,
     );
@@ -1213,7 +1213,7 @@ describe('<DataGrid /> - Layout & warnings', () => {
     }
 
     render(
-      <div style={{ height: 'auto', width: 1584 }}>
+      <div style={{ height: 300, width: 1584 }}>
         <DataGrid
           rows={[{ id: 1 }]}
           columns={[

packages/x-data-grid/src/tests/pagination.DataGrid.test.tsx

@@ -730,12 +730,14 @@ describe('<DataGrid /> - Pagination', () => {
     ];
     expect(() => {
       const { setProps } = render(
-        <DataGrid
-          columns={columns}
-          rows={rows}
-          initialState={{ pagination: { paginationModel: { pageSize: 2, page: 1 } } }}
-          pageSizeOptions={[2]}
-        />,
+        <div style={{ width: 300, height: 300 }}>
+          <DataGrid
+            columns={columns}
+            rows={rows}
+            initialState={{ pagination: { paginationModel: { pageSize: 2, page: 1 } } }}
+            pageSizeOptions={[2]}
+          />
+        </div>,
       );
       setProps({ rows: rows.slice(0, 2) });
     }).not.to.throw();

test/karma.conf.js

@@ -3,7 +3,7 @@ const webpack = require('webpack');
 
 const CI = Boolean(process.env.CI);
 
-process.env.CHROME_BIN = chromium.executablePath();
+process.env.CHROME_BIN ||= chromium.executablePath();
 
 // Karma configuration
 module.exports = function setKarmaConfig(config) {