Skip to content

fix(ui): hide the native tabline in CodeDiff views - #441

Closed
richardgill wants to merge 5 commits into
esmuellert:mainfrom
richardgill:remove-diff-path-header
Closed

fix(ui): hide the native tabline in CodeDiff views#441
richardgill wants to merge 5 commits into
esmuellert:mainfrom
richardgill:remove-diff-path-header

Conversation

@richardgill

@richardgill richardgill commented Jul 16, 2026

Copy link
Copy Markdown
Collaborator

Hide Neovim’s native tabline while a CodeDiff tab is active, restoring the user’s setting on codediff leave or cleanup.

image

Fixes: #191

Before

before.mp4

After

pr441-after-branch-only.mp4

@richardgill
richardgill marked this pull request as draft July 17, 2026 10:53
@richardgill
richardgill marked this pull request as ready for review July 18, 2026 09:17
@richardgill richardgill added the Size/M Medium - Write logic, add tests (one evening) label Jul 22, 2026
@richardgill
richardgill requested a review from esmuellert as a code owner July 23, 2026 12:02

@esmuellert esmuellert left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

I tried it by myself. Basically my understanding of the issue is current behavior is always showing the tab regardless of user's settings. Now it becomes always not showing tab. I feel both are not idea ways if we want to optimize, because I use lualine which expects a tab at all time, but the latest fix will not show it for me. Is it possible to honor user's settings about tab? If their settings show tab, we should have it, if not, we don't have it

Image

@richardgill

Copy link
Copy Markdown
Collaborator Author

Thanks, you’re right. CodeDiff should honor the user’s showtabline setting. Setting mine to 0 solves the issue, so I’ll close this PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Size/M Medium - Write logic, add tests (one evening)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Can't hide the buffer tabs during diff review

2 participants