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

Search highlighting across multiple buffers #1091

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

ChoppinBlockParty
Copy link

Provide a pilot version as PoC.
No cross-buffer highlighting for substitution in this version.
On the way improve overall performance of highlighting
by removing run-at-time and introducing window-end
with update equal t.

Search pattern in Vim is highlighted in every visible buffer, this does not sound like a big deal, however it is very useful in many small situations. This feature is also addressed in issue #805.

Please, try it out, feedback is highly appreciated. I have tested it quickly on few buffers with some large content as well (like ~20k) worked well, I think the previous version with timer was lagging much more. After some feedback and a few days of real life testing, I will polish this one to be ready for merge.

@ChoppinBlockParty ChoppinBlockParty force-pushed the all-windows-highlighting-ex-search branch from a4cf6d7 to c82ec11 Compare October 8, 2018 14:35
@ChoppinBlockParty ChoppinBlockParty force-pushed the all-windows-highlighting-ex-search branch 2 times, most recently from aa78a71 to 8c871f1 Compare June 15, 2020 12:41
Provide a pilot version as PoC.
No cross-buffer highlighting for substitution in this version.
On the way improve overall performance of highlighting
by removing `run-at-time` and introducing `window-end`
with `update` equal `t`.
@ChoppinBlockParty ChoppinBlockParty force-pushed the all-windows-highlighting-ex-search branch from 8c871f1 to e03c0ea Compare June 15, 2020 12:46
@ChoppinBlockParty
Copy link
Author

Hello Guys, I have been using this one for more than a year it is working well. Can we consider merging it?

@TheBB
Copy link
Member

TheBB commented Jun 15, 2020

Sorry. :-)

I'll have a look this week hopefully.

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