Skip to content

Add setting to open the review panel in a split - #51

Merged
philphilphil merged 1 commit into
philphilphil:mainfrom
matejherman2:panel-location-setting
Oct 15, 2026
Merged

philphilphil merged 1 commit into
philphilphil:mainfrom
matejherman2:panel-location-setting

Conversation

@matejherman2

Copy link
Copy Markdown
Contributor

Thanks for the plugin, I use it daily for reviewing AI suggestions.

The review panel always opens in the right sidebar. When the sidebar already holds other views (calendar, outline, properties…), the panel ends up in a small stacked pane, and long threads are hard to read. I always ended up dragging it into the main area by hand.

This adds a setting under Editing & display:

  • Open review panel in: Right sidebar (default, current behaviour) / New split beside the note

With split, openReviewPanel uses workspace.getLeaf("split", "vertical") instead of getRightLeaf(false), so the panel gets a full-height column next to the note. An already open panel is still just revealed wherever it is, so nothing changes for people who keep it open.

Details:

  • panelLocation: "sidebar" | "split" in settings, coerced in loadSettings like replyDateStyle (no migration needed, default keeps existing behaviour)
  • documented in the Settings section of CLAUDE.md
  • npm run build, npm test and eslint src pass

Related: #43 also touches openReviewPanel (split beside the note in pop-out windows). The two should combine easily: this setting would pick the main-window target, #43 the per-window lookup. Happy to rebase on top of it if that one lands first.

🤖 Generated with Claude Code

New "Open review panel in" dropdown (Editing & display): "Right sidebar"
(default, unchanged behaviour) or "New split beside the note". In the
right sidebar the panel shares a narrow stack with other side views;
a split gives it a full-height column next to the note being reviewed.
An already open panel is still revealed wherever it is.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@philphilphil
philphilphil merged commit 7c1cee0 into philphilphil:main Oct 15, 2026
1 check passed
@philphilphil

Copy link
Copy Markdown
Owner

thank you for your PR, will release soon

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.

2 participants