Broken zsh completion menu

[anuvyklack]

General information

• Operating system: Pop!_OS 21.04
• ZSH framework: none
• ZSH version: 5.8-6
• ZVM version: 82f00ea

Basic examination

• I have read through the README page
• I have the latest version of zsh-vi-mode
• I have tested with another terminal program: kitty, gnome-terminal, tilix

Problem description

On the latest commit (82f00ea) zsh completion menu broke inside tmux. Outside of tmux it works.

Reproduction steps

1. setopt menu_complete
2. enter tmux, print any command (cd for example) and press Tab

simplescreenrecorder-2021-09-07_19.20.12.mov

[jeffreytse]

Hi @anuvyklack

Thanks for your reporting. I will work on this issue as soon as possible.

Thanks and Regards

[jeffreytse]

Hi @anuvyklack

I can't reproduce this issue in my environment. Could you please try this version 1549325, and feedback me the result?

Thanks and Regards

[anuvyklack]

1549325 works fine.

[jeffreytse]

@anuvyklack Could you please provide me the information by command echo $TERM

[anuvyklack]

tmux-256color

[jeffreytse]

@anuvyklack Could you please provide me a minimum .zshrc file which can reproduce this issue to have further check?

[stefanboca]

@jeffreytse I also had this issue. Reverting to 1549325 fixed it, and I managed to trace it down to the following changes:

-  zvm_update_cursor
+  if [[ -n $TMUX ]]; then
+    zvm_update_cursor
+    # Fix display is not updated in the terminal of InteliJ IDE
+    zle redisplay
+  fi

Reverting this change fixed the issue for me.

[jeffreytse]

@stefanboca Thanks for your reaching out, I have confirmed the reason of this issue, but I am still exploring a good way to fix this issue, at the same time, keep the compatibility of the terminal in IntelliJ IDE. Could you provide me a minimum .zshrc file which can reproduce this issue to have a further check? Thanks and Regards

[stefanboca]

@jeffreytse
My minimal .zshrc is

export ZSH="/home/user/.oh-my-zsh"
plugins=(zsh-vi-mode)
source $ZSH/oh-my-zsh.sh

If I load the plugin directly without loading Oh My Zsh, though, the problem disappears:

export ZSH="/home/user/.oh-my-zsh"
source $ZSH/custom/plugins/zsh-vi-mode/zsh-vi-mode.plugin.zsh

[jeffreytse]

Hi @anuvyklack @stefanboca

Thanks for your patience, now this issue has been addressed and fixed. Please update your plugin to the latest version and try again.

Thanks and Regards

[anuvyklack]

Hello, @jeffreytse.

I'm sorry, but for me still doesn't work. The same: all commits after c1480b4 are broken.

[jeffreytse]

Hi @anuvyklack

For the temporary solution, you can comment the line L3152 to disable redisplay as below:

 if [[ -n $TMUX ]]; then
    zvm_update_cursor
    # Fix display is not updated in the terminal of IntelliJ IDE.
    # We should update display only when the last widget isn't a
    # completion widget
    #
    # PLEASE COMMENT BELOW LINE
    #
    # [[ $LASTWIDGET =~ 'complet' ]] || zle redisplay
 fi

Thanks & Regards

[weirongxu]

Same problem here.
My environment:

General information

Linux

• Operating system: Linux 5.9.16-1-MANJARO x86_64
• ZSH framework: none
• ZSH version: 5.8 (x86_64-pc-linux-gnu)
• ZVM version: 7878372
• TMUX version: 3.2a
• Terminal emulator: wezterm 20220101-133340-7edc5b5a

---

MacOS

• Operating system: Darwin 20.6.0 x86_64
• ZSH framework: none
• ZSH version: 5.8.1 (x86_64-apple-darwin20.6.0)
• ZVM version: 7878372
• TMUX version: 3.2a
• Terminal emulator: iTerm2 Build 3.4.15

Reproduction steps

1. ~/.zshrc

   autoload -U +X compinit && compinit
   zstyle ':completion:*' menu yes select
   source /Users/raidou/repos/zsh-vi-mode/zsh-vi-mode.zsh

2. enter tmux
3. input ls and press tab more than twice

[michaelorr]

I am also running into this issue. My environment is structurally the same as @weirongxu 's MacOS setup.
Thank you so much for the workaround. Commenting out the zle redisplay fixes it for me. I'm leaving a comment just so you can gauge the impact and priority for this fix. Also, I'd hate to miss out on future updates because I'm no longer tracking upstream. Thanks so much for all your hard work on this plugin!

[michaelorr]

I have pushed up a PR which attempts to fix this issue:
#191

I cannot test in IntelliJ and would love if someone were able to verify the behavior there.

[michaelorr]

Barring any further bug reports here, I think this is fixed by #191

[michaelorr]

Thank you everyone for the helpful diagnostic information above that led to the fix and thank you to @jeffreytse for maintaining this project and merging my commit!