Skip to content
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

Don't bind any zle-* methods #206

Closed
wants to merge 1 commit into from
Closed

Don't bind any zle-* methods #206

wants to merge 1 commit into from

Conversation

docwhat
Copy link
Contributor

@docwhat docwhat commented Nov 5, 2016

It seems like all the zle-* methods are special and shouldn't be
monkeyed with.

Specifically zle-isearch-update and friends. Binding that widget
caused history-incremental-pattern-search to stop working.

Fixes zsh-users/zsh-syntax-highlighting#387

It seems like all the zle-* methods are special and shouldn't be
monkeyed with.

Specifically `zle-isearch-update` and friends. Binding that widget
caused `history-incremental-pattern-search` to stop working.

Fixes zsh-users/zsh-syntax-highlighting#387
docwhat added a commit to docwhat/dotfiles that referenced this pull request Nov 5, 2016
Figured out what was causing the problem(s).

I reported it at zsh-users/zsh-syntax-highlighting#387

I submitted a pull request at zsh-users/zsh-autosuggestions#206
docwhat added a commit to docwhat/dotfiles that referenced this pull request Nov 5, 2016
Figured out what was causing the problem(s).

I reported it at zsh-users/zsh-syntax-highlighting#387

I submitted a pull request at zsh-users/zsh-autosuggestions#206
@docwhat
Copy link
Contributor Author

docwhat commented Nov 5, 2016

I apologize. I meant to target the develop the target branch. Can you fix it after-the-fact or do I need to submit a new pull request? Feel free to re-implement it yourself, I don't care about credit, just working code. ;-)

@ericfreese
Copy link
Member

ericfreese commented Jan 23, 2017

I will look more into this PR later, but for now you may be able to get the desired behavior by adding zle-* to the list of ZSH_AUTOSUGGEST_IGNORE_WIDGETS in your local config.

% ZSH_AUTOSUGGEST_IGNORE_WIDGETS=($ZSH_AUTOSUGGEST_IGNORE_WIDGETS zle-\*)

@ericfreese
Copy link
Member

Cherry picked onto develop (0c940e7). Thank you!

@ericfreese ericfreese closed this Feb 18, 2017
@ericfreese ericfreese mentioned this pull request Feb 19, 2017
13 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants