Wrap user-defined widgets

[danielshahaf]

_zsh_highlight_bind_widgets uses the following code to decide which widgets to wrap:

  for cur_widget in ${${(f)"$(builtin zle -la)"}:#(.*|_*|orig-*|run-help|which-command|beep|yank-pop)}; do
    case $widgets[$cur_widget] in

      # Already rebound event: do nothing.
      user:$cur_widget|user:_zsh_highlight_widget_*);;

User-defined widgets, such as edit-command-line or even a trivial f() {}; zle -N f (which both fall into the first branch of that case pattern), should be wrapped to recompute highlighting like other widgets.

[danielshahaf]

Right now, the code looks like this:

zsh-syntax-highlighting/zsh-syntax-highlighting.zsh

Lines 206 to 212 in 82f5eb1

	# Already rebound event: do nothing.
	user:$cur_widget|user:_zsh_highlight_widget_*);;
	# User defined widget: override and rebind old one with prefix "orig-".
	user:*) eval "zle -N orig-$cur_widget ${widgets[$cur_widget]#*:}; \
	_zsh_highlight_widget_$cur_widget() { _zsh_highlight_call_widget orig-$cur_widget \"\$@\" }; \
	zle -N $cur_widget _zsh_highlight_widget_$cur_widget";;

  206         # Already rebound event: do nothing.
  207         user:$cur_widget|user:_zsh_highlight_widget_*);;
  208             
  209         # User defined widget: override and rebind old one with prefix "orig-".
  210         user:*) eval "zle -N orig-$cur_widget ${widgets[$cur_widget]#*:}; \
  211                       _zsh_highlight_widget_$cur_widget() { _zsh_highlight_call_widget orig-$cur_widget \"\$@\" }; \
  212                       zle -N $cur_widget _zsh_highlight_widget_$cur_widget";;

Why does it behave differently for zle -N foo foo (takes the user:$cur_widget case) and zle -N foo bar (takes the user:* case)?

[danielshahaf]

Fixed some time ago in 487b122.