Skip to content

Followup to #415 #423

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

Closed
danielshahaf opened this issue Mar 30, 2017 · 2 comments
Closed

Followup to #415 #423

danielshahaf opened this issue Mar 30, 2017 · 2 comments
Milestone

Comments

@danielshahaf
Copy link
Member

Breaking out the following comment since #415 is already merged and thus can't be reopened:

#415 (comment)

/cc @docwhat @m0vie

@docwhat
Copy link
Contributor

docwhat commented Apr 5, 2017

So this is to fix the documentation as per @m0vie's comment?

If you're using `history-incremental-search-backward` (by default bound to <kbd>Ctrl+R</kbd>
in zsh's emacs keymap) then it works with _zsh version 5.3 and newer_.

If you're using `history-incremental-pattern-search-backward`, then syntax highlighting works
in _zsh 5.3.2 and newer_ due to [a bug in zsh](http://www.zsh.org/cgi-bin/mla/redirect?WORKERNUMBER=40285).

Under zsh 5.2 and older, the zsh-default [underlining][zshzle-Character-Highlighting]
of the matched portion of the buffer remains available, but zsh-syntax-highlighting's
additional highlighting is unavailable.  (Those versions of zsh do not provide
enough information to allow computing the highlighting correctly.)

history-incremental-search-backward or history-incremental-pattern-search-backward doesn't matter, we don't check for that. Its 5.3.2 and newer in all cases.

Did I misunderstand what the $ISEARCHMATCH_ACTIVE does? I thought it was indicating that the history-incremental-pattern-search-* was being used?

@m0vie
Copy link
Contributor

m0vie commented Apr 5, 2017

ISEARCHMATCH_ACTIVE is just a flag that indicates that you typed something in Ctrl-R that matches. It is unrelated to the widget which is used.

In reality, currently, with the zsh_highlight__pat_static_bug condition the check for the flag is redundant, since there are no versions >= 5.3.2 which don't have it. But i'd leave that as it is, should the bug condition ever change.

I simply propose this as the FAQ test:

Highlighting the command line during an incremental history search, which is
bound by default to <kbd>Ctrl+R</kbd> in zsh's emacs keymap requires zsh 5.3.2
or newer.

m0vie added a commit to m0vie/zsh-syntax-highlighting that referenced this issue Jun 3, 2017
m0vie added a commit to m0vie/zsh-syntax-highlighting that referenced this issue Jun 3, 2017
m0vie added a commit to m0vie/zsh-syntax-highlighting that referenced this issue Jul 30, 2017
m0vie added a commit to m0vie/zsh-syntax-highlighting that referenced this issue Jul 30, 2017
m0vie added a commit to m0vie/zsh-syntax-highlighting that referenced this issue Jul 30, 2017
m0vie added a commit to m0vie/zsh-syntax-highlighting that referenced this issue Jul 30, 2017
m0vie added a commit to m0vie/zsh-syntax-highlighting that referenced this issue Jul 30, 2017
brandon-fryslie pushed a commit to brandon-fryslie/zsh-syntax-highlighting that referenced this issue Dec 21, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants