add css rule to keep header visible when maxHeight is set

Author: Ian Grant

packages/@adobe/spectrum-css-temp/components/table/index.css

@@ -44,6 +44,7 @@ svg.spectrum-Table-sortedIcon {
   border-left-style: solid;
   border-right-width: 1px;
   border-right-style: solid;
+  flex: 0 0 auto;
 }
 .spectrum-Table-headCellContents {
   display: inline-block;

packages/@react-spectrum/table/chromatic/TableView.chromatic.tsx

@@ -198,6 +198,10 @@ export const NestedColumns = Template().bind({});
 NestedColumns.storyName = 'nested columns';
 NestedColumns.args = {columns: nestedColumns, items};
 
+export const MaxHeight = Template.bind({});
+MaxHeight.storyName = 'maxHeight defined';
+MaxHeight.args = {columns, items, maxHeight: 'size-100'};
+
 export const Empty = EmptyTemplate().bind({});
 Empty.storyName = 'empty table';
 Empty.args = {};

packages/@react-spectrum/table/test/Table.test.js

@@ -33,6 +33,7 @@ import {TableWithBreadcrumbs} from '../stories/Table.stories';
 import {TextField} from '@react-spectrum/textfield';
 import {theme} from '@react-spectrum/theme-default';
 import {typeText} from '@react-spectrum/test-utils';
+import {useAsyncList} from '@react-stately/data';
 import userEvent from '@testing-library/user-event';
 
 let columns = [
@@ -120,7 +121,10 @@ describe('TableView', function () {
   });
 
   beforeEach(() => {
-    jest.spyOn(window, 'requestAnimationFrame').mockImplementation(cb => cb());
+    jest.spyOn(window, 'requestAnimationFrame').mockImplementation(cb => {
+      cb(0);
+      return 0;
+    });
   });
 
   afterEach(() => {