-
-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Support zellij floating pane via --popup (new name for --tmux) #4145
base: master
Are you sure you want to change the base?
Conversation
Love this, it works super well, pretty much how I'd expect! |
Thanks for testing it. Here's why this is still marked as a draft. It currently doesn't work well with non-percent width and height values. Zellij lacks options to open floating windows at the center of the screen, unlike tmux (i.e. Also, as I mentioned in the original discussion, the lack of an option to run a program in a blocking mode forces us to rely on another wrapper script using a temporary named pipe, and it complicates the implementation, which I find unfortunate. Another problem I noticed is that Zellij somehow fails to process
And it makes us have silly code branches in our code, and I'm not so happy with it. |
I did see that TODO so I hadn't bothered testing that part in particular. I appreciate you putting time into a fairly niche request anyway, especially given the time of year. |
I'm going to ask if they're willing to make the floating pane start at the center of the screen by default. It's a little strange that it no longer starts at the center once a custom width or height is given. Given that zellij-org/zellij#3447 was reported 6 months ago, but no progress has been made, I don't think we can wait for all the issues to be resolved. |
Another thing I would like to see from Zellij is an option to disable the border around the floating pane, because fzf has its own |
Replaces `fzf_bin = "fzf-tmux"` as "--tmux" is the preferred method for both fzf and skim. Fixed getting the correct width from tmux window/pane for accurate "flex" layout calculation based on % in setting (default to 50%). This commit also perpares us better for the upcoming "--popup" option in fzf in order to support zellij popups. Ref: junegunn/fzf#4145
No description provided.