Skip to content

Conversation

@narsaynorath
Copy link
Member

@narsaynorath narsaynorath commented Nov 13, 2025

Always tries to show all of the telemetry icons on the right panel in the default split, not the minimum right side width.

@narsaynorath narsaynorath requested a review from a team as a code owner November 13, 2025 16:26
@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Nov 13, 2025
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.

Bug: Panel Sizing Inconsistency Hides Icons

The max constraint for the left panel uses width - MIN_RIGHT_WIDTH (370px for right panel) but the defaultSplit calculation ensures 506px for the right panel to show telemetry icons. This inconsistency allows users to drag the divider and shrink the right panel below the size needed to display telemetry icons, defeating the purpose of the changes. The max should use the same calculation as defaultSplit.

static/app/views/explore/metrics/metricPanel/sideBySideOrientation.tsx#L101-L102

min: MIN_LEFT_WIDTH,
max: width - MIN_RIGHT_WIDTH,

Fix in Cursor Fix in Web


@narsaynorath narsaynorath enabled auto-merge (squash) November 13, 2025 16:54
@narsaynorath narsaynorath merged commit b4d6bee into master Nov 13, 2025
49 checks passed
@narsaynorath narsaynorath deleted the nar/fix/tracemetrics-update-split-panel-ratio-by-nav-collapsed branch November 13, 2025 17:00
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