From b2f329d61a1302ff318f753ab342d016b2a8264c Mon Sep 17 00:00:00 2001 From: Anton Standrik Date: Tue, 2 Sep 2025 18:59:13 +0300 Subject: [PATCH 1/3] fix: query tables styles --- .../QueryResultTable/QueryResultTable.tsx | 20 +++++++++---------- .../ResizeableDataTable.scss | 2 -- 2 files changed, 9 insertions(+), 13 deletions(-) diff --git a/src/components/QueryResultTable/QueryResultTable.tsx b/src/components/QueryResultTable/QueryResultTable.tsx index 4a4ad6da50..a7d7c1f7f7 100644 --- a/src/components/QueryResultTable/QueryResultTable.tsx +++ b/src/components/QueryResultTable/QueryResultTable.tsx @@ -105,16 +105,14 @@ export const QueryResultTable = (props: QueryResultTableProps) => { } return ( -
- -
+ ); }; diff --git a/src/components/ResizeableDataTable/ResizeableDataTable.scss b/src/components/ResizeableDataTable/ResizeableDataTable.scss index 762c6a6a8a..3366464842 100644 --- a/src/components/ResizeableDataTable/ResizeableDataTable.scss +++ b/src/components/ResizeableDataTable/ResizeableDataTable.scss @@ -1,6 +1,4 @@ .ydb-resizeable-data-table { - display: flex; - width: max-content; // padding for easier resize of the last column From 877708ffc45d822e1837809470d2dad8f04bbcc4 Mon Sep 17 00:00:00 2001 From: Anton Standrik Date: Wed, 3 Sep 2025 13:44:23 +0300 Subject: [PATCH 2/3] fix: fullScreen --- src/components/Fullscreen/Fullscreen.tsx | 7 +++++++ .../components/ResultSetsViewer/ResultSetsViewer.tsx | 10 ---------- 2 files changed, 7 insertions(+), 10 deletions(-) diff --git a/src/components/Fullscreen/Fullscreen.tsx b/src/components/Fullscreen/Fullscreen.tsx index 2c87070716..f223a33a1c 100644 --- a/src/components/Fullscreen/Fullscreen.tsx +++ b/src/components/Fullscreen/Fullscreen.tsx @@ -61,6 +61,13 @@ function Fullscreen({children, className}: FullscreenProps) { } else { ref.current?.appendChild(container); } + // Trigger recalculation for components relying on window resize + // Dispatch after DOM re-parent to ensure correct measurements + requestAnimationFrame(() => { + requestAnimationFrame(() => { + window.dispatchEvent(new Event('resize')); + }); + }); } }, [container, isFullscreen]); diff --git a/src/containers/Tenant/Query/QueryResult/components/ResultSetsViewer/ResultSetsViewer.tsx b/src/containers/Tenant/Query/QueryResult/components/ResultSetsViewer/ResultSetsViewer.tsx index f50c081852..4be7d62ad8 100644 --- a/src/containers/Tenant/Query/QueryResult/components/ResultSetsViewer/ResultSetsViewer.tsx +++ b/src/containers/Tenant/Query/QueryResult/components/ResultSetsViewer/ResultSetsViewer.tsx @@ -6,7 +6,6 @@ import {Flex, Tab, TabList, TabProvider, Text} from '@gravity-ui/uikit'; import {QueryResultTable} from '../../../../../../components/QueryResultTable'; import type {ParsedResultSet} from '../../../../../../types/store/query'; import {cn} from '../../../../../../utils/cn'; -import {useTypedSelector} from '../../../../../../utils/hooks'; import {QueryResultError} from '../QueryResultError/QueryResultError'; import './ResultSetsViewer.scss'; @@ -23,20 +22,11 @@ interface ResultSetsViewerProps { export function ResultSetsViewer(props: ResultSetsViewerProps) { const {selectedResultSet, setSelectedResultSet, resultSets, error} = props; - const isFullscreen = useTypedSelector((state) => state.fullscreen); const scrollRef = React.useRef(null); const currentResult = resultSets?.[selectedResultSet]; - React.useEffect(() => { - //this is needed to trigger data-table recount visible rows - if (isFullscreen) { - const resizeEvent = new Event('resize'); - scrollRef.current?.dispatchEvent(resizeEvent); - } - }, [isFullscreen]); - const renderTabs = () => { return ( From 8fbcf582615b810b276cc3c974039f8a3c7e073a Mon Sep 17 00:00:00 2001 From: Anton Standrik Date: Wed, 3 Sep 2025 14:07:37 +0300 Subject: [PATCH 3/3] fix: one request --- src/components/Fullscreen/Fullscreen.tsx | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/src/components/Fullscreen/Fullscreen.tsx b/src/components/Fullscreen/Fullscreen.tsx index f223a33a1c..14490b188b 100644 --- a/src/components/Fullscreen/Fullscreen.tsx +++ b/src/components/Fullscreen/Fullscreen.tsx @@ -62,11 +62,9 @@ function Fullscreen({children, className}: FullscreenProps) { ref.current?.appendChild(container); } // Trigger recalculation for components relying on window resize - // Dispatch after DOM re-parent to ensure correct measurements + // Dispatch after DOM repaint to ensure correct measurements requestAnimationFrame(() => { - requestAnimationFrame(() => { - window.dispatchEvent(new Event('resize')); - }); + window.dispatchEvent(new Event('resize')); }); } }, [container, isFullscreen]);