Skip to content

Conversation

@ibetitsmike
Copy link
Contributor

@ibetitsmike ibetitsmike commented Dec 15, 2025

Keep RightSidebar width stable when switching Costs ↔ Review (no collapse-to-0 flash).

  • Use a single persisted container width across both tabs.
  • Remove AIView ↔ RightSidebar tab-sync plumbing; switching tabs now only swaps content.
  • Avoid Costs/Context auto-collapse oscillation when the sidebar is wider than the default width.

Validation:

  • make static-check

Generated with mux β€’ Model: openai:gpt-5.2 β€’ Thinking: xhigh

Notify AIView of tab changes synchronously (click/keybind) so Review width doesn't momentarily compute to 0px.

---
_Generated with `mux` β€’ Model: `openai:gpt-5.2` β€’ Thinking: `xhigh`_
@chatgpt-codex-connector
Copy link

Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits.
Repo admins can enable using credits for code reviews in their settings.

Stop changing RightSidebar width when switching Costs ↔ Review by using a single persisted width for the container.

---
_Generated with `mux` β€’ Model: `openai:gpt-5.2` β€’ Thinking: `xhigh`_
Avoid auto-collapse oscillation on Costs/Context when the sidebar is wider than the default width.

---
_Generated with `mux` β€’ Model: `openai:gpt-5.2` β€’ Thinking: `xhigh`_
@ibetitsmike ibetitsmike added this pull request to the merge queue Dec 15, 2025
Merged via the queue into main with commit 42385f0 Dec 15, 2025
19 of 20 checks passed
@ibetitsmike ibetitsmike deleted the mike/fix-rightsidebar-tab-width-jank branch December 15, 2025 14:59
github-merge-queue bot pushed a commit that referenced this pull request Dec 16, 2025
The sidebar width is now shared between Costs and Review tabs (from
#1172), but the resize handle was still only visible on Review. Show the
handle on both tabs so users can resize regardless of which tab is
active.

---
_Generated with `mux` β€’ Model: `anthropic:claude-opus-4-5` β€’ Thinking:
`high`_
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant