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

fixes #8783 add easymotionSearchLines #9100

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

HenryTSZ
Copy link
Contributor

What this PR does / why we need it:

fixes #8783 add easymotionSearchLines

Which issue(s) this PR fixes

#8783

Special notes for your reviewer:

@J-Fields
Copy link
Member

Does https://github.com/easymotion/vim-easymotion have an equivalent setting?

@J-Fields
Copy link
Member

And is there any good reason we shouldn't just stop at the screen's edge, rather than make this configurable or hardcoded to 100?

@HenryTSZ
Copy link
Contributor Author

Does https://github.com/easymotion/vim-easymotion have an equivalent setting?

Vim EasyMotion itself does not directly provide a configuration item for setting the number of rows to be crossed during search

@HenryTSZ
Copy link
Contributor Author

And is there any good reason we shouldn't just stop at the screen's edge, rather than make this configurable or hardcoded to 100?

Firstly, we usually only see the content within the current screen range

Secondly, if using a vertical display, the screen display range may exceed 100 lines

@kaiguang
Copy link

Secondly, if using a vertical display, the screen display range may exceed 100 lines

Besides the large display, if I have a fold with more than 100 lines, I cannot jump over that fold area with EasyMotion. Being able to change the 100 to something larger will be helpful.

Also, an idea solution would be jumping anywhere within the screen range, and I don't really to need to search within the folded lines.

1 <cursor_here>
2
3...100 <folded_lines>
101
102
103                   <-- cannot jump here

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.

Is there any way to set a limit for EasyMotion?
3 participants