Skip to content

Commit

Permalink
chore(stacktrace): Make source map tooltip aligned (#82016)
Browse files Browse the repository at this point in the history
this has annoyed me since forever

before: 


![image](https://github.com/user-attachments/assets/33733ee9-646b-4949-a18d-6a29e16b6028)


after: (notice the tooltip is aligned with the stack trace file path
now)


![image](https://github.com/user-attachments/assets/2e199d45-1126-4878-84ea-41e11d50130c)


(This changes applies to both the old and new issue details pages)
  • Loading branch information
MichaelSun48 authored Dec 13, 2024
1 parent 847b8d6 commit 29fcc33
Showing 1 changed file with 8 additions and 9 deletions.
17 changes: 8 additions & 9 deletions static/app/components/events/interfaces/frame/deprecatedLine.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -364,15 +364,13 @@ export class DeprecatedLine extends Component<Props, State> {
{this.isExpandable() ? <InteractionStateLayer /> : null}
<DefaultLineTitleWrapper isInAppFrame={data.inApp}>
<LeftLineTitle>
<div>
{this.renderLeadHint()}
<DefaultTitle
frame={data}
platform={this.props.platform ?? 'other'}
isHoverPreviewed={isHoverPreviewed}
meta={this.props.frameMeta}
/>
</div>
{this.renderLeadHint()}
<DefaultTitle
frame={data}
platform={this.props.platform ?? 'other'}
isHoverPreviewed={isHoverPreviewed}
meta={this.props.frameMeta}
/>
</LeftLineTitle>
</DefaultLineTitleWrapper>
<DefaultLineTagWrapper>
Expand Down Expand Up @@ -508,6 +506,7 @@ const DefaultLineTitleWrapper = styled('div')<{isInAppFrame: boolean}>`
const LeftLineTitle = styled('div')`
display: flex;
align-items: center;
gap: ${space(0.25)};
`;

const RepeatedContent = styled(LeftLineTitle)`
Expand Down

0 comments on commit 29fcc33

Please sign in to comment.