-
-
Notifications
You must be signed in to change notification settings - Fork 678
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Feature Request] Support use of Zellij with Neovim smart splits plugin #2434
Comments
Adding extra context here. I reached out in the matrix channel and there is a caveat:
Maybe it would be possible for I'll also copy my table here for convenience. I have a Lua API wrapper around multiplexer backends, here's the support table I made for Zellij:
For the |
Yes I think not supporting multiple sessions would be fine, and plugin integration would still be super useful for most users. |
Maybe this issue will help. #967 |
It looks like that issue is already resolved. Anyway, it's only somewhat related. A way to get the currently focused Pane ID is the only thing preventing
|
Adding this action to get the current pane id should not be a problem really, just needs someone to pick it up. |
I am not familiar with |
For full/better support, we'd also need a way to check if that pane is at an edge of the screen, and a way to tell if the pane is zoomed/maximized, but we can do rudimentary support with just a way to get the current pane ID. To check if we're at the edge, we can try moving to a new pane, and if the same pane ID is returned, then we're at an edge. It's a bit of a hack and looks a little weird since the cursor will flash, but it works. See the table I support posted above. |
I've got an active pr to implement swap tab - #2242, it already does a check if pane is at an edge, I can include that in my pr as rudimentary support :) |
I've published a pr for this, want to have some feedbacks before I continue |
QQs:
|
|
Thanks, :) FYI - I am having an active pr to handle wrap around panes: Say you have one tab with 3 panes in zellij - A|B|C, and the C is current one, then when you move to the right, zellij will automatically focus the leftmost one which is A. For this feature request - having 3 separated APIs(paneId, at the edge and is_in_session) makes more sense, I will continue the rest of code changes |
I've added the support to know if a pane is on the edge by giving direction. For |
yeah this is more for my side, it can be implemented by checking if the current pane ID returns a value or not, in my plugin. I don’t think we need anything special from zellij for this. |
I’d really like to see this supported because having 3 separate sets of keybinds for switching windows, zellij panes and nvim splits is quite inconvenient. |
Hey @mrjones2014 - just released a patch version (0.40.1) that among other things includes:
I think these two should allow you to perform the integration? Also just note: you can always look at the |
This is the pane ID of the current pane, right? I still need to use If that's possible now, then yeah I think we should be able to integrate with Zellij! |
To your questions, in order: yes and yes :) Happy hacking! |
This neovim plugin supports tmux and I was wondering if it could support zellij as well: https://github.com/mrjones2014/smart-splits.nvim
I asked on the plugin github page and the author says he would need command line support for a few things: mrjones2014/smart-splits.nvim#111
Have a look and see if this is something you would consider implementing, to increase adoption of zellij in the neovim community. Thank you!
The text was updated successfully, but these errors were encountered: