feat: redesign diff view with horizontal layout and new tab options

[ThomasK33]

Improved Diff View Layout and Configuration

This PR refactors the diff view functionality to provide more flexible layout options and better tab management:

• Added a new layout option to replace the previous vertical_split boolean, supporting both "vertical" and "horizontal" layouts
• Replaced open_in_current_tab with open_in_new_tab for more intuitive configuration
• Removed redundant options auto_close_on_accept and show_diff_stats that were no longer used
• Added proper type annotations for configuration options
• Implemented improved window management when opening diffs in new tabs
• Added terminal window preservation when opening diffs in new tabs
• Improved window option handling to maintain consistent appearance across different layouts
• Enhanced cleanup logic to properly handle both tab-based and window-based diff views
• Updated tests to reflect the new configuration options

These changes make the diff view more customizable while simplifying the configuration interface. Users can now choose between vertical or horizontal layouts and easily open diffs in new tabs while maintaining access to the terminal.

[ThomasK33]

• feat: add focus_after_send option for terminal behavior #118
• feat: add working directory control for Claude terminal #117
• feat: add Shift+Enter keybinding for new line in terminal #116
• feat: add PartialClaudeCodeConfig type for better configuration handling #115
• feat: add option to control behavior when rejecting new file diffs #114
• feat: redesign diff view with horizontal layout and new tab options #111 👈 (View in Graphite)
• feat: add Claude Haiku model and update type annotations #110
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

[alex35mil]

@ThomasK33 thanks for the fix! The only issue I noticed so far is that after the accept/reject, the new tab remains opened (just not active), so I end up with bunch of tabs opened - one per each diff.

[alex35mil]

Actually, if I reject (with a keymap, not via CC cli menu) - the new tab is closed, but Claude Code split is gone as well.

[alex35mil]

Also, after accepting, the inner terminal width becomes wider and goes off-screen until I close all diff tabs and refocus the claude code window.

[ThomasK33]

@ThomasK33 thanks for the fix! The only issue I noticed so far is that after the accept/reject, the new tab remains opened (just not active), so I end up with bunch of tabs opened - one per each diff.

Got it, thanks @alex35mil. I've removed the tab and diff closing parts and am waiting for Claude Code to send a tab close tool call. When accepting or denying a diff now, it seems to close the tab and diff correctly.

[alex35mil]

@ThomasK33 Tested the update. Everything seems to be fixed except I need to refocus the Claude window to fix its width after accept/reject with keymap - it still goes off screen after these actions.

Also, another idea for the diff in a new tab - having an option to hide the Claude window in the new tab with diff, thus there's more screen estate for reviewing the diff. I find myself not looking at the Claude output during the review, so it's a waste of screen space.

[alex35mil]

Oh, and I just noticed a bunch of unnamed buffers created after the accept/reject dance in the new tab.

[S1M0N38]

Is it possible to have an option to disable the Neovim diff view entirely (so to just use the diff provided directly in claudecode)? Maybe something like

diff_opts = {
  enabled = true
  -- other diff opts ...
}

The new field diff_opts.enabled in opts could be a boolean with true as the default for backward compatibility.

I've decided to comment here because I've noticed this PR is about a diff view redesign, and I think this is something related to that. If you prefer, I can open a separate issue.

[ThomasK33]

Is it possible to have an option to disable the Neovim diff view entirely (so to just use the diff provided directly in claudecode)? Maybe something like

diff_opts = {

  enabled = true

  -- other diff opts ...

}

The new field diff_opts.enabled in opts could be a boolean with true as the default for backward compatibility.

I've decided to comment here because I've noticed this PR is about a diff view redesign, and I think this is something related to that. If you prefer, I can open a separate issue.

Nope, won't be adding that. You can configure that in Claude Code itself. See: #45 (comment)

[lcmen]

Any chance to get this merged?

[ThomasK33]

Merge activity

• Sep 10, 3:36 PM UTC: A user started a stack merge that includes this pull request via Graphite.
• Sep 10, 3:36 PM UTC: @ThomasK33 merged this pull request with Graphite.

[lcmen]

Thank you!