From 89e86a041e0ceb8498d3d2d860e1fc4540a56eda Mon Sep 17 00:00:00 2001 From: novykh Date: Mon, 13 Jul 2026 12:10:46 +0300 Subject: [PATCH 1/4] Address large data table review findings --- src/components/table/body/header/cell.js | 3 +- src/components/table/body/index.js | 69 +++++----- src/components/table/body/index.test.js | 18 ++- src/components/table/body/measureElement.js | 29 +--- .../table/body/measureElement.test.js | 25 ++-- src/components/table/body/row.js | 124 +++++++----------- .../table/body/rowMountController.js | 13 +- .../table/components/overflowTooltip.js | 49 ++++--- .../table/components/overflowTooltip.test.js | 33 ++++- src/components/table/createLargeDataSource.js | 56 ++++---- .../table/createLargeDataSource.test.js | 34 ++++- src/components/table/header/actions/index.js | 9 +- .../table/headerOverflowTooltip.test.js | 64 ++++++--- src/components/table/helpers/columnValue.js | 8 ++ src/components/table/helpers/downloadCsv.js | 10 +- src/components/table/largeData.js | 22 +--- src/components/table/largeData.test.js | 15 +-- src/components/table/largeDataSelection.js | 42 ++++-- .../table/largeDataSelection.test.js | 7 +- src/components/table/table.js | 86 +++--------- src/components/table/useLargeData.js | 111 ++++++++++++++++ src/components/templates/mixins/alignItems.js | 2 +- src/components/templates/mixins/flex.js | 2 +- .../templates/mixins/justifyContent.js | 2 +- .../navigation/sortable/item.test.js | 12 +- 25 files changed, 484 insertions(+), 361 deletions(-) create mode 100644 src/components/table/helpers/columnValue.js create mode 100644 src/components/table/useLargeData.js diff --git a/src/components/table/body/header/cell.js b/src/components/table/body/header/cell.js index 4db524c26..7e84838b4 100644 --- a/src/components/table/body/header/cell.js +++ b/src/components/table/body/header/cell.js @@ -11,6 +11,7 @@ import Info from "./info" import Filter from "./filter" import useSortableHeader from "./sortableHeader" import DragHandle from "./dragHandle" +import { overflowTooltipAttribute } from "../../components/overflowTooltip" const Label = styled(Text)` width: 100%; @@ -122,7 +123,7 @@ const BodyHeaderCell = ({