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

Add per view search location and total matches to statusline. #11700

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

Conversation

useche
Copy link
Contributor

@useche useche commented Sep 14, 2024

This patch changes search_impl to calculate the index of the current match and the total matches in the search. It also adds a new option in the status line to show this information.

@Axlefublr
Copy link
Contributor

love this, was clearly missing

@kirawi kirawi added A-gui Area: Helix gui improvements A-command Area: Commands labels Sep 28, 2024
helix-term/src/commands.rs Outdated Show resolved Hide resolved
@david-crespo
Copy link
Contributor

Wonderful, I love this. No noticeable lag on an M1 Mac in a 21k line file with 56k matches for ". I think a cap on the number of matches would be fine — 99.9% of the time I would not be hitting the cap.

book/src/editor.md Outdated Show resolved Hide resolved
@useche useche force-pushed the search_position branch 3 times, most recently from 8ddba74 to e668b57 Compare October 6, 2024 19:00
This patch changes `search_impl` to calculate the index of the current
match and the total matches in the search. It also adds a new option in
the status line to show this information.
The new option `max-matches` can be set to `"none"` or an integer that
specifies that maximum number of matches. It's set by default to `100`.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-command Area: Commands A-gui Area: Helix gui improvements
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants