Skip to content

fix(playground): avoid anthropic reasoning menu reshaping#11729

Merged
yfrigui2 merged 4 commits intoArize-ai:mainfrom
pandego:fix/11530-shapeshifting-menu
Feb 26, 2026
Merged

fix(playground): avoid anthropic reasoning menu reshaping#11729
yfrigui2 merged 4 commits intoArize-ai:mainfrom
pandego:fix/11530-shapeshifting-menu

Conversation

@pandego
Copy link
Contributor

@pandego pandego commented Feb 25, 2026

fixes #11530

The Anthropic reasoning config menu was changing shape when toggling Thinking Enabled because the Budget Tokens field was conditionally mounted/unmounted.

This keeps the field mounted at all times and only toggles its disabled/required state. That keeps the menu layout stable while preserving existing behavior for enabling reasoning.

Validation

  • npm run lint
  • npm run typecheck
  • npm run test
  • npm run fmt

Note

Low Risk
Low risk UI-only change that adjusts a popover container sizing prop; no business logic or data handling is affected.

Overview
Prevents the model-parameters configuration popover from resizing by switching the inner View from minWidth="386px" to a fixed width="386px", keeping the menu layout stable as fields appear/disappear.

Written by Cursor Bugbot for commit 3bb45c6. This will update automatically on new commits. Configure here.

@pandego pandego requested a review from a team as a code owner February 25, 2026 13:06
@github-project-automation github-project-automation bot moved this to 📘 Todo in phoenix Feb 25, 2026
@dosubot dosubot bot added the size:M This PR changes 30-99 lines, ignoring generated files. label Feb 25, 2026
@dosubot dosubot bot added size:XS This PR changes 0-9 lines, ignoring generated files. and removed size:M This PR changes 30-99 lines, ignoring generated files. labels Feb 26, 2026
@yfrigui2
Copy link
Contributor

hey @pandego, thanks for looking at this! I think we should update the modal to constrain its width instead, since I don't think it should be growing so wide.

@yfrigui2 yfrigui2 enabled auto-merge (squash) February 26, 2026 15:08
@github-project-automation github-project-automation bot moved this from 📘 Todo to 👍 Approved in phoenix Feb 26, 2026
@yfrigui2 yfrigui2 merged commit 623d722 into Arize-ai:main Feb 26, 2026
27 of 33 checks passed
@github-project-automation github-project-automation bot moved this from 👍 Approved to ✅ Done in phoenix Feb 26, 2026
mikeldking pushed a commit that referenced this pull request Feb 27, 2026
* fix(playground): keep anthropic reasoning panel layout stable

* constrain width of playground model config popover

---------

Co-authored-by: Yasmine Frigui <yfrigui@arize.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

[bug] shapeshifting menu

2 participants