-
Notifications
You must be signed in to change notification settings - Fork 63
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
buggy in Tmux #279
Comments
Do you have passthrough enabled? |
Yes, this is my tmux config:
|
What terminal do you use, do you have ueberzug setup (if you use it)? |
I use Wezterm and also tried Kitty, both of which have the same issue. (I don’t need ueberzug for these two, right?) |
no |
The second issue might be related to neovim and tmux themselves. check neovim/neovim#32609. It just happened after I installed image.nvim. However, I still wish we could address issue 1. Changing the command string into a list of arguments would work. |
I tried using ueberzugpp with foot, ghostty and alacritty and it worked perfectly fine. When I tried using ghostty with the kitty backend the view jumps a bit if the cursor gets close to the image. that is something that shouldn't happen, because it doesn't with ueberzug @stevalkr |
It also happens without tmux, in KGP terminal! What you explained doesn't happen to me though, so it might be correct, that its related to neovim and tmux themselves |
@UnaTried, could you reproduce the first issue I mentioned? In my fish configuration, I have an alias defined as follows: # functions/tmux.fish
function tmux
function __fish_set_user_var
if type -q base64
printf "\033]1337;SetUserVar=%s=%s\007" "$argv[1]" (echo -n "$argv[2]" | base64)
end
end
__fish_set_user_var IS_TMUX true
command tmux $argv
__fish_set_user_var IS_TMUX false
end And this would break If there’s no specific reason to execute commands in a shell, I’ll open a pull request to address this issue. image.nvim/lua/image/utils/tmux.lua Lines 3 to 7 in 6ffafab
|
lua/image/utils/tmux.lua
file, tmux commands are executed through a shell. However, it’s quite common to create aliases for tmux to execute default commands, layouts, or SetUserVars, among other things. Unfortunately, this practice can break all the functionalities if any output is written to the stdout. It might be better to not use a shell by default.can\’t\ find\ window:\ \'0\'
is created under the current working directory. This issue persists even with (1) fixed, and I haven’t found the reason yet.The text was updated successfully, but these errors were encountered: