Skip to content

feat(trace): bring back dashed backgrounds#69321

Merged
JonasBa merged 5 commits into
masterfrom
jb/trace/queries-buttons
Apr 23, 2024
Merged

feat(trace): bring back dashed backgrounds#69321
JonasBa merged 5 commits into
masterfrom
jb/trace/queries-buttons

Conversation

@JonasBa

@JonasBa JonasBa commented Apr 19, 2024

Copy link
Copy Markdown
Contributor

No description provided.

@codecov

codecov Bot commented Apr 23, 2024

Copy link
Copy Markdown

Bundle Report

Changes will increase total bundle size by 17.12kB ⬆️

Bundle name Size Change
app-webpack-bundle-array-push 26.38MB 17.12kB ⬆️

@JonasBa JonasBa force-pushed the jb/trace/queries-buttons branch from f65395d to 9bb432d Compare April 23, 2024 18:54
});

for (let i = 0; i < this.columns.list.column_refs.length; i++) {
const span_bar = this.span_bars[i];

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All of these methods got consolidated into the appropriate drawXXX calls, hence why the diff got huge. It was unmaintainable to keep them all inline

@JonasBa JonasBa requested a review from Abdkhan14 April 23, 2024 18:56
@JonasBa JonasBa marked this pull request as ready for review April 23, 2024 18:56
@JonasBa JonasBa requested a review from a team April 23, 2024 18:56
@JonasBa JonasBa merged commit b3cc1ab into master Apr 23, 2024
@JonasBa JonasBa deleted the jb/trace/queries-buttons branch April 23, 2024 20:37
@sentry

sentry Bot commented Apr 28, 2024

Copy link
Copy Markdown
Contributor

Suspect Issues

This pull request was deployed and Sentry observed the following issues:

  • ‼️ TypeError: Failed to initialize NumberFormat since used feature is not supported in the linked ICU version Trace(app/views/performance/newTraceDetails/trace) View Issue
  • ‼️ Error: Autogrouped node must have span children computeAutogroupedBarSegments(app/views/perform... View Issue
  • ‼️ TypeError: Node must be a transaction or span onNodeZoomIn(app/views/performance/newTraceDeta... View Issue

Did you find this useful? React with a 👍 or 👎

@github-actions github-actions Bot locked and limited conversation to collaborators May 17, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants