Skip to content

feat(detectors): Add connected line graph to mobile build visualization#113012

Open
mtopo27 wants to merge 2 commits intomasterfrom
mtopo27/new-mobile-builds-monitor-graphic
Open

feat(detectors): Add connected line graph to mobile build visualization#113012
mtopo27 wants to merge 2 commits intomasterfrom
mtopo27/new-mobile-builds-monitor-graphic

Conversation

@mtopo27
Copy link
Copy Markdown
Contributor

@mtopo27 mtopo27 commented Apr 15, 2026

Add a line-graph polyline connecting all data points in the MobileBuildVisualization SVG, giving the chart a connected appearance instead of isolated dots.

Circles are filled with the parent container's background color so the polyline appears cleanly interrupted at each data point rather than showing through underneath.

Also moves the preprod_size_analysis detector type entry higher in the list and wires it up to the new visualization (previously visualization: null).

Add a polyline connecting all data points in MobileBuildVisualization
and fill circles with the background color so the line appears cleanly
interrupted at each dot.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Apr 15, 2026
@mtopo27 mtopo27 marked this pull request as ready for review April 15, 2026 00:02
@mtopo27 mtopo27 requested a review from a team as a code owner April 15, 2026 00:02
@sentry
Copy link
Copy Markdown
Contributor

sentry bot commented Apr 15, 2026

Sentry Snapshot Testing

Name Added Removed Modified Renamed Unchanged Status
sentry-frontend
sentry-frontend
0 0 0 0 204 ✅ Unchanged

Copy link
Copy Markdown
Contributor

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 4928035. Configure here.

Comment thread static/app/views/detectors/components/detectorTypeForm.tsx Outdated
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

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