Skip to content

Fit side panel height to content#5689

Open
viktorashi wants to merge 5 commits into
jesseduffield:masterfrom
viktorashi:master
Open

Fit side panel height to content#5689
viktorashi wants to merge 5 commits into
jesseduffield:masterfrom
viktorashi:master

Conversation

@viktorashi

@viktorashi viktorashi commented Jun 11, 2026

Copy link
Copy Markdown

PR Description

I was getting annoyed that the entire branches side-panel was taking up my entire screen, even though I only had like 2 branches in my repo:

lazygit-demo-pic

So I made the "acordion" thingy size down to fit the content, and leave room for the other tabs that might have more rows (like Files, mostly for example):

lazygit-demo-pic-2

It also works for the other ones

Captură de ecran 2026-06-11 115510

This how a fully empty one looks now

Captură de ecran 2026-06-11 114724

Please check if the PR fulfills these requirements

  • Cheatsheets are up-to-date (run go generate ./...)
  • Code has been formatted (see here)
  • Tests have been added/updated (see here for the integration test guide)
  • Text is internationalised (see [here] (https://github.com/jesseduffield/lazygit/blob/master/CONTRIBUTING.md#internationalisation)) - not applicable
  • If a new UserConfig entry was added, make sure it can be hot-reloaded (see here)
  • Docs have been updated if necessary
  • You've read through your own file changes for silly mistakes etc. - Not the best code quality, I'm sorry I just wanted the feature and had Antigravity have a go at it. I don't wanna waste your time with AI PR's but I geniuenly think it'd be useful for folks

vic-stan added 5 commits June 11, 2026 11:46
I followed the formatting guide from "contributing" and apparently it
hadn't been run for a while, with the default config at least...
@viktorashi

Copy link
Copy Markdown
Author

now that I look at it, it might be a little too much motion for some people, when changing between tabs.
I was working on a fitAllSidePanelsHeightToContent flag as well to help with that, but it's not going that great in its dev

@stefanhaller

Copy link
Copy Markdown
Collaborator

@viktorashi Sorry, but this repo is no longer accepting PRs.

The idea of not wasting so much space for empty panel sounds interesting though; I can put it on my list of things to think about.

@viktorashi

viktorashi commented Jun 11, 2026

Copy link
Copy Markdown
Author

@viktorashi Sorry, but this repo is no longer accepting PRs.

The idea of not wasting so much space for empty panel sounds interesting though; I can put it on my list of things to think about.

oh noo, is it related to too many AI slop PR's being filed in the last year or so?
If so, im sorry for slightly contributing to that today

But no matter what, thanks a ton for working so much on this amazing project. and Jesse as well, I havent seen any annoucemeents or stuff like that on this repo tbh

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.

3 participants