Skip to content

fix(ai-drawer): Accomodate pretty/raw button in json viewer component#106549

Merged
priscilawebdev merged 1 commit intomasterfrom
priscila/fix/accomodate-pretty-raw-button-in-multiline-text-wrapper-monospace
Jan 20, 2026
Merged

fix(ai-drawer): Accomodate pretty/raw button in json viewer component#106549
priscilawebdev merged 1 commit intomasterfrom
priscila/fix/accomodate-pretty-raw-button-in-multiline-text-wrapper-monospace

Conversation

@priscilawebdev
Copy link
Member

@priscilawebdev priscilawebdev commented Jan 20, 2026

The MultilineTextWrapperMonospace component requires a min-height while
MultilineTextWrapper does not due to font size differences:

  • MultilineTextWrapper uses default 14px font, which naturally renders
    ~36px tall with padding, providing adequate space for the absolutely
    positioned Pretty/Raw segmented control

  • MultilineTextWrapperMonospace uses smaller 12px monospace font for code
    display, which would naturally render shorter (~32px). Without min-height,
    short JSON values creates cramped layouts

The min-height calculation ensures the container is always tall enough to
accommodate the segmented control (form height + spacing) regardless of
content length or font size.

Before After
image image

closes: https://linear.app/getsentry/issue/TET-1729/output-prettyraw-button-spacing-looks-off

@linear
Copy link

linear bot commented Jan 20, 2026

@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Jan 20, 2026
@priscilawebdev priscilawebdev marked this pull request as ready for review January 20, 2026 07:36
@priscilawebdev priscilawebdev requested a review from a team as a code owner January 20, 2026 07:36
@priscilawebdev priscilawebdev requested a review from a team January 20, 2026 07:36
@priscilawebdev priscilawebdev merged commit 0335cd3 into master Jan 20, 2026
55 checks passed
@priscilawebdev priscilawebdev deleted the priscila/fix/accomodate-pretty-raw-button-in-multiline-text-wrapper-monospace branch January 20, 2026 10:43
@github-actions github-actions bot locked and limited conversation to collaborators Feb 4, 2026
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