Skip to content

Let the inspector panel resize freely#1676

Merged
datlechin merged 1 commit into
mainfrom
feat/free-resize-inspector
Jun 13, 2026
Merged

Let the inspector panel resize freely#1676
datlechin merged 1 commit into
mainfrom
feat/free-resize-inspector

Conversation

@datlechin

Copy link
Copy Markdown
Member

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: dropped inspectorSplitItem.maximumThickness = 400. The default is NSSplitViewItemUnspecifiedDimension, so there is no upper bound now.
  • Kept minimumThickness = 270 so the panel can't shrink to an unusable width.
  • No effect on window min-size logic: recomputeWindowMinSize() only reads minimums.
  • The divider drag and existing 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.

@datlechin datlechin merged commit c4b6dce into main Jun 13, 2026
3 of 4 checks passed
@datlechin datlechin deleted the feat/free-resize-inspector branch June 13, 2026 12:12

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 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

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge 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 👍 / 👎.

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