From 2ba3f25afe11345665b07382919e186bd55720ec Mon Sep 17 00:00:00 2001 From: Jorge Padilla Date: Thu, 8 Feb 2024 16:14:37 -0500 Subject: [PATCH] remove prop --- web/src/components/RunDetailTrace/TracePanel.tsx | 3 --- web/src/components/RunDetailTrace/Visualization.tsx | 13 ++----------- .../components/Timeline/TimelineV2.tsx | 7 ++++--- 3 files changed, 6 insertions(+), 17 deletions(-) diff --git a/web/src/components/RunDetailTrace/TracePanel.tsx b/web/src/components/RunDetailTrace/TracePanel.tsx index 8eb6f50e9b..ed958e3f09 100644 --- a/web/src/components/RunDetailTrace/TracePanel.tsx +++ b/web/src/components/RunDetailTrace/TracePanel.tsx @@ -11,8 +11,6 @@ import Visualization from './Visualization'; import {FillPanel} from '../ResizablePanels'; import SkipTraceCollectionInfo from '../SkipTraceCollectionInfo'; -const HEADER_HEIGHT = 210; - type TProps = { run: TestRun; testId: string; @@ -55,7 +53,6 @@ const TracePanel = ({run, testId, runEvents, skipTraceCollection}: TProps) => { runState={run.state} trace={run.trace} type={visualizationType} - containerHeight={window.innerHeight - HEADER_HEIGHT} /> diff --git a/web/src/components/RunDetailTrace/Visualization.tsx b/web/src/components/RunDetailTrace/Visualization.tsx index c9d2c7fde0..b0d4fc8419 100644 --- a/web/src/components/RunDetailTrace/Visualization.tsx +++ b/web/src/components/RunDetailTrace/Visualization.tsx @@ -14,7 +14,6 @@ import {VisualizationType} from './RunDetailTrace'; import TraceDAG from './TraceDAG'; interface IProps { - containerHeight: number; isDAGDisabled: boolean; runEvents: TestRunEvent[]; runState: TTestRunState; @@ -22,15 +21,7 @@ interface IProps { type: VisualizationType; } -const Visualization = ({ - containerHeight, - isDAGDisabled, - runEvents, - runState, - trace, - trace: {spans, rootSpan}, - type, -}: IProps) => { +const Visualization = ({isDAGDisabled, runEvents, runState, trace, trace: {spans, rootSpan}, type}: IProps) => { const dispatch = useAppDispatch(); const selectedSpan = useAppSelector(TraceSelectors.selectSelectedSpan); @@ -54,7 +45,7 @@ const Visualization = ({ return type === VisualizationType.Dag && !isDAGDisabled ? ( ) : ( - + ); }; diff --git a/web/src/components/Visualization/components/Timeline/TimelineV2.tsx b/web/src/components/Visualization/components/Timeline/TimelineV2.tsx index 9cbcc5dc8c..3258ba4023 100644 --- a/web/src/components/Visualization/components/Timeline/TimelineV2.tsx +++ b/web/src/components/Visualization/components/Timeline/TimelineV2.tsx @@ -8,13 +8,14 @@ import SpanNodeFactory from './SpanNodeFactoryV2'; import TimelineProvider from './Timeline.provider'; import * as S from './TimelineV2.styled'; +const HEADER_HEIGHT = 210; + export interface IProps { - containerHeight: number; nodeType: NodeTypesEnum; spans: Span[]; } -const Timeline = ({containerHeight, nodeType, spans}: IProps) => { +const Timeline = ({nodeType, spans}: IProps) => { const listRef = useRef(null); const nodes = useMemo(() => TimelineModel(spans, nodeType), [spans, nodeType]); @@ -23,7 +24,7 @@ const Timeline = ({containerHeight, nodeType, spans}: IProps) => {