Let the inspector panel resize freely#1676
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 4e73c05ba2
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| @@ -211,7 +211,6 @@ internal final class MainSplitViewController: NSSplitViewController, InspectorVi | |||
| inspectorSplitItem = NSSplitViewItem(inspectorWithViewController: inspectorHosting) | |||
| inspectorSplitItem.canCollapse = true | |||
| inspectorSplitItem.minimumThickness = 270 | |||
There was a problem hiding this comment.
Explicitly clear the inspector maximum thickness
Because this item is created with NSSplitViewItem(inspectorWithViewController:), AppKit initializes the inspector pane with the standard inspector min/max width of 270 points and makes it non-resizable by default. This patch removes the old maximumThickness = 400 override but only reassigns minimumThickness, so in windows using this controller the inspector will clamp back to 270 instead of becoming freely resizable; set maximumThickness explicitly to an unbounded/unspecified value after creating the inspector item.
Useful? React with 👍 / 👎.
What
Removes the fixed 400pt max-width cap on the right inspector panel so it can be resized freely by dragging its divider.
Why
The inspector was hard-capped between 270 and 400 points. Users with wider content (long values, AI chat) couldn't widen it past 400.
Changes
MainSplitViewController: droppedinspectorSplitItem.maximumThickness = 400. The default isNSSplitViewItemUnspecifiedDimension, so there is no upper bound now.minimumThickness = 270so the panel can't shrink to an unusable width.recomputeWindowMinSize()only reads minimums.autosaveName("com.TablePro.mainSplit") persist the chosen width across launches.Notes
The CHANGELOG entry for this already landed on main via #1674, so it's not repeated here.