Skip to content

Commit d3b87d2

Browse files
committed
adjust traceViewRef
1 parent 1bd38e2 commit d3b87d2

File tree

3 files changed

+6
-33
lines changed

3 files changed

+6
-33
lines changed

src/sentry/static/sentry/app/views/organizationEventsV2/transactionView/minimap.tsx

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,6 @@ const TIME_AXIS_HEIGHT = 30;
2626
const VIEW_HANDLE_HEIGHT = 20;
2727

2828
type PropType = {
29-
traceViewRef: React.RefObject<HTMLDivElement>;
3029
minimapInteractiveRef: React.RefObject<HTMLDivElement>;
3130
dragProps: DragManagerChildrenProps;
3231
trace: ParsedTraceType;

src/sentry/static/sentry/app/views/organizationEventsV2/transactionView/spanTree.tsx

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -23,12 +23,13 @@ type RenderedSpanTree = {
2323
};
2424

2525
type PropType = {
26-
traceViewRef: React.RefObject<HTMLDivElement>;
2726
trace: ParsedTraceType;
2827
dragProps: DragManagerChildrenProps;
2928
};
3029

3130
class SpanTree extends React.Component<PropType> {
31+
traceViewRef = React.createRef<HTMLDivElement>();
32+
3233
shouldComponentUpdate(nextProps: PropType) {
3334
if (nextProps.dragProps.isDragging) {
3435
return false;
@@ -172,8 +173,8 @@ class SpanTree extends React.Component<PropType> {
172173
) : null;
173174

174175
return (
175-
<DividerHandlerManager.Provider interactiveLayerRef={this.props.traceViewRef}>
176-
<TraceViewContainer innerRef={this.props.traceViewRef}>
176+
<DividerHandlerManager.Provider interactiveLayerRef={this.traceViewRef}>
177+
<TraceViewContainer innerRef={this.traceViewRef}>
177178
{spanTree}
178179
{hiddenSpansMessage}
179180
</TraceViewContainer>

src/sentry/static/sentry/app/views/organizationEventsV2/transactionView/traceView.tsx

Lines changed: 2 additions & 29 deletions
Original file line numberDiff line numberDiff line change
@@ -20,35 +20,12 @@ type PropType = {
2020
event: Readonly<SentryEvent>;
2121
};
2222

23-
type StateType = {
24-
renderMinimap: boolean;
25-
};
26-
27-
class TraceView extends React.Component<PropType, StateType> {
23+
class TraceView extends React.Component<PropType> {
2824
minimapInteractiveRef = React.createRef<HTMLDivElement>();
29-
traceViewRef = React.createRef<HTMLDivElement>();
30-
31-
state: StateType = {
32-
renderMinimap: false,
33-
};
34-
35-
componentDidMount() {
36-
if (this.traceViewRef.current) {
37-
// eslint-disable-next-line react/no-did-mount-set-state
38-
this.setState({
39-
renderMinimap: true,
40-
});
41-
}
42-
}
4325

4426
renderMinimap = (dragProps: DragManagerChildrenProps, parsedTrace: ParsedTraceType) => {
45-
if (!this.state.renderMinimap) {
46-
return null;
47-
}
48-
4927
return (
5028
<TraceViewMinimap
51-
traceViewRef={this.traceViewRef}
5229
minimapInteractiveRef={this.minimapInteractiveRef}
5330
dragProps={dragProps}
5431
trace={parsedTrace}
@@ -176,11 +153,7 @@ class TraceView extends React.Component<PropType, StateType> {
176153
return (
177154
<React.Fragment>
178155
{this.renderMinimap(dragProps, parsedTrace)}
179-
<SpanTree
180-
traceViewRef={this.traceViewRef}
181-
trace={parsedTrace}
182-
dragProps={dragProps}
183-
/>
156+
<SpanTree trace={parsedTrace} dragProps={dragProps} />
184157
</React.Fragment>
185158
);
186159
}}

0 commit comments

Comments
 (0)