Skip to content

feat(view-hierarchy): Allow forcing absolute positioning#107507

Open
timfish wants to merge 2 commits intogetsentry:masterfrom
timfish:timfish/feat/view-hierarchy-absolute
Open

feat(view-hierarchy): Allow forcing absolute positioning#107507
timfish wants to merge 2 commits intogetsentry:masterfrom
timfish:timfish/feat/view-hierarchy-absolute

Conversation

@timfish
Copy link
Contributor

@timfish timfish commented Feb 3, 2026

Currently relative/absolute positioning for View Hierarchy wireframes is always relative unless the platform matches a predefined list:

getHierarchyDimensions(
hierarchy,
['flutter', 'dart-flutter'].includes(platform ?? '')
),

This PR allows overriding the platform defaults via the root positioning property in the hierarchy data.

@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Feb 3, 2026
@timfish
Copy link
Contributor Author

timfish commented Feb 5, 2026

I think the CI failures are related to permissions because this is on my fork. Let me know if you want me to push this to the Sentry repo (if I have permissions!).

@timfish timfish marked this pull request as ready for review February 5, 2026 15:01
@timfish timfish requested a review from a team as a code owner February 5, 2026 15:01
Copy link
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.

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

@Lms24 Lms24 added the Trigger: getsentry tests Once code is reviewed: apply label to PR to trigger getsentry tests label Feb 5, 2026
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 Trigger: getsentry tests Once code is reviewed: apply label to PR to trigger getsentry tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants