diff --git a/design-system/pkg/src/list-view/ListView.tsx b/design-system/pkg/src/list-view/ListView.tsx index 6e226d09f..4af87cbd8 100644 --- a/design-system/pkg/src/list-view/ListView.tsx +++ b/design-system/pkg/src/list-view/ListView.tsx @@ -66,9 +66,9 @@ function useListLayout( let layout = useMemo( () => new ListViewLayout({ - estimatedRowHeight: ROW_HEIGHTS[density][scale], + estimatedRowHeight: + overflowMode === 'wrap' ? undefined : ROW_HEIGHTS[density][scale], }), - // eslint-disable-next-line react-hooks/exhaustive-deps [scale, density, overflowMode] ); diff --git a/design-system/pkg/src/table/InsertionIndicator.tsx b/design-system/pkg/src/table/InsertionIndicator.tsx index 8c6244506..386cdc54f 100644 --- a/design-system/pkg/src/table/InsertionIndicator.tsx +++ b/design-system/pkg/src/table/InsertionIndicator.tsx @@ -25,6 +25,8 @@ export function InsertionIndicator(props: InsertionIndicatorProps) { ); assert(!!dropState, 'dropState is not defined.'); + // if the indicator is rendered dnd hooks are defined + // eslint-disable-next-line react-compiler/react-compiler let { dropIndicatorProps } = dragAndDropHooks.useDropIndicator( props, dropState, diff --git a/design-system/pkg/src/table/TableView.tsx b/design-system/pkg/src/table/TableView.tsx index be4888667..d1a5ebc7e 100644 --- a/design-system/pkg/src/table/TableView.tsx +++ b/design-system/pkg/src/table/TableView.tsx @@ -207,8 +207,6 @@ export function TableView( estimatedHeadingHeight: props.overflowMode === 'wrap' ? DEFAULT_HEADER_HEIGHT : undefined, }), - // don't recompute when state.collection changes, only used for initial value - // eslint-disable-next-line react-hooks/exhaustive-deps [props.overflowMode, density] ); diff --git a/design-system/pkg/src/tooltip/test/TooltipTrigger.test.tsx b/design-system/pkg/src/tooltip/test/TooltipTrigger.test.tsx index 56bc5548a..5325342a5 100644 --- a/design-system/pkg/src/tooltip/test/TooltipTrigger.test.tsx +++ b/design-system/pkg/src/tooltip/test/TooltipTrigger.test.tsx @@ -386,6 +386,7 @@ describe('tooltip/TooltipTrigger', () => { }); // can't get this to work with timers + // eslint-disable-next-line jest/no-commented-out-tests // it('opens for hover', async () => { // let { getByRole, getByLabelText } = renderWithProvider( //