Optionally extract Worktrees into a separate side window and make Status window optional#5326
Conversation
7a7e6a9 to
c424430
Compare
Up to standards ✅🟢 Issues
|
| Metric | Results |
|---|---|
| Complexity | 19 |
| Duplication | 2 |
NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes. Give us feedback
c424430 to
12e1236
Compare
7f924cb to
ab448e5
Compare
|
I like this. I would use for sure. Working with LLMs worktrees are super important |
|
I understand the wish, but I don't like the approach taken here, which is not flexible enough. I opened #5702 which allows reconfiguring the side panels with much more flexibility; please test and/or give feedback on that. |
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
ab448e5 to
d31f3f6
Compare
|
Superseded by #5702; closing. |
Oh thank you so much! And sorry, just noticed your reply. Will give it a go, being able to configure everything sounds even better than my approach 👍🏼 |
PR Description
Disclaimer: this PR is completely clauded and as I don't have experience with go, I can't really guarantee code quality, so apologies in advance.
Motivation
I'm a heavy worktree user and one thing I wanted improved in the app is visibility of all the worktrees and the currently selected one. Right now the worktree menu is buried in the second window group as a separate tab. What I always wanted is to have a dedicated worktree window group.
While thinking about it I realised if I was to add a separate group, the total group count would be 6 which can be a bit too tight on smaller screens. Plus not everybody is using worktrees and for some it would be a useless change that occupies precious terminal real estate.
So, I decided to add 2 config flags, one for showing the worktree group as a separate group, second one is for hiding the very first status group to free up some space in case user doesn't care about that group. Both are
falseby default.Implementation (claude generated)
Two new independent gui config options:
When both are enabled, the layout becomes: Worktrees[1] → Files[2] → Branches[3] → Commits[4] → Stash[5] — preserving the default 5-key jump binding.
When worktreesInSeparateGroup is enabled without hiding status, there are 6 windows. The jump-to-block controller auto-extends keybindings (adding key "6") so no manual config change is needed.
Key changes:
Screenshots
Please check if the PR fulfills these requirements
go generate ./...)