E52D feat(dashboards): scaffold mobile session health module by DominikB2014 · Pull Request #107167 · getsentry/sentry · GitHub
[go: up one dir, main page]

Skip to content

Conversation

@DominikB2014
Copy link
Contributor
@DominikB2014 DominikB2014 commented Jan 28, 2026

Scaffold basic mobile health module with what is currently available in dashboards. Visibility of the dashboard version is behind a flag
image

@DominikB2014 DominikB2014 requested a review from a team as a code owner January 28, 2026 17:42
@linear
Copy link
linear bot commented Jan 28, 2026

@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Jan 28, 2026
Copy link
Contributor
@cursor cursor bot left a comment

Choose a reason for hiding this comm F43F ent

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

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

Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.

],
},
],
4
Copy link
Contributor

Choose a reason for hiding this comment

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

Dashboard layout has gap due to incorrect y position

Medium Severity

THIRD_ROW_WIDGETS is positioned at y=4 but SECOND_ROW_WIDGETS ends at row 2 (y=1 with h=2). This creates an empty row 3 in the dashboard layout. The y parameter passed to spaceWidgetsEquallyOnRow should be 3 instead of 4 to ensure continuous widget placement without gaps.

Fix in Cursor Fix in Web

@DominikB2014 DominikB2014 merged commit 8389c84 into master Jan 28, 2026
53 checks passed
@DominikB2014 DominikB2014 deleted the dominikbuszowiecki/browse-250-scaffold-basic-mobile-session-health-module branch January 28, 2026 21:28
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.

3 participants

0