fix: review

Author: astandrik

src/components/QueryResultTable/QueryResultTable.tsx

@@ -61,16 +61,14 @@ type RenderCellArgs = {row: KeyValueRow; columnName: string};
 type RenderCell = (args: RenderCellArgs) => React.ReactNode;
 
 interface CreateRenderCellParams {
-    activeCellRef: React.RefObject<ActiveCellState>;
+    activeCell: ActiveCellState;
     dispatch: React.Dispatch<ActiveCellAction>;
 }
 
-function createRenderCell({activeCellRef, dispatch}: CreateRenderCellParams): RenderCell {
+function createRenderCell({activeCell, dispatch}: CreateRenderCellParams): RenderCell {
     return ({row, columnName}: RenderCellArgs) => {
         const isActive = Boolean(
-            activeCellRef.current &&
-                activeCellRef.current.row === row &&
-                activeCellRef.current.columnName === columnName,
+            activeCell && activeCell.row === row && activeCell.columnName === columnName,
         );
 
         const value = row[columnName];
@@ -147,13 +145,10 @@ export const QueryResultTable = (props: QueryResultTableProps) => {
 
     const [activeCell, dispatch] = React.useReducer(activeCellReducer, null);
 
-    const activeCellRef = React.useRef<ActiveCellState>(null);
-
-    React.useEffect(() => {
-        activeCellRef.current = activeCell;
-    }, [activeCell]);
-
-    const renderCell = React.useMemo(() => createRenderCell({activeCellRef, dispatch}), [dispatch]);
+    const renderCell = React.useMemo(
+        () => createRenderCell({activeCell, dispatch}),
+        [activeCell, dispatch],
+    );
 
     const preparedColumns = React.useMemo(() => {
         return columns

src/containers/Tenant/Diagnostics/Network/NodeNetwork/NodeNetwork.tsx

@@ -2,6 +2,7 @@ import React from 'react';
 
 import {EFlag} from '../../../../../types/api/enums';
 import {cn} from '../../../../../utils/cn';
+import type {NodeTooltipData} from '../NodeTooltipPopup/NodeTooltipPopup';
 
 import './NodeNetwork.scss';
 
@@ -22,13 +23,6 @@ function getNodeModifier(connected = 0, capacity = 0) {
 
 function noop() {}
 
-interface NodeTooltipData {
-    nodeId: number | string;
-    connected?: number;
-    capacity?: number;
-    rack: string;
-}
-
 interface NodeNetworkProps {
     onMouseEnter?: (node: HTMLDivElement, data: NodeTooltipData) => void;
     onMouseLeave?: () => void;

src/containers/Tenant/Diagnostics/Network/Nodes/Nodes.tsx

@@ -5,18 +5,12 @@ import type {TNetNodeInfo, TNetNodePeerInfo} from '../../../../../types/api/netI
 import {cn} from '../../../../../utils/cn';
 import {useWithProblemsQueryParam} from '../../../../../utils/hooks/useWithProblemsQueryParam';
 import {NodeNetwork} from '../NodeNetwork/NodeNetwork';
+import type {NodeTooltipData} from '../NodeTooltipPopup/NodeTooltipPopup';
 import i18n from '../i18n';
 import {getConnectedNodesCount, groupNodesByField} from '../utils';
 
 const b = cn('network');
 
-interface NodeTooltipData {
-    nodeId: number | string;
-    connected?: number;
-    capacity?: number;
-    rack: string;
-}
-
 export interface NodesProps {
     nodes: Record<string, (TNetNodeInfo | TNetNodePeerInfo)[]>;
     isRight?: boolean;