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

AceJump stops highlighting the correct line after adding new lines to the file #373

Closed
albrtbc opened this issue May 13, 2021 · 1 comment
Labels

Comments

@albrtbc
Copy link

albrtbc commented May 13, 2021

Describe the bug
When you open a new file and use AceJump it works correctly. But if you add a new line and search again it seems like it doesn't refresh the document and highlights the words as if they were on a previous line, so it marks a line higher than it really is.

The more lines you add, the more the gap between the highlighting and the word you are looking for. In addition, everything new you add is not found.

To Reproduce
Steps to reproduce the behavior:

  1. Open a file with text.
  2. Add a new line at the top of the file
  3. Search a word that's in the second line
  4. Highlights the first line

Expected behavior
It should match the highlighted line with the current word.

Screenshots
Here I was searching for 'aaa' which is in the second line and it's highlighting the first line.
image

Desktop (please complete the following information):

  • OS: Windows
  • IDE and version: Rider 2021.1.2
  • AceJump version: 3.8.0

Additional context
Looks like it caches the file and once you write new lines / stuff it doesn't refresh it so it's always looking for the old version line.

@albrtbc albrtbc added the bug label May 13, 2021
@breandan
Copy link
Collaborator

Hi @albrtbc, thank you for reporting this issue! I am a little surprised it slipped through undetected. Indeed, it was caused by a stale cache. I have added a test and confirmed the issue is no longer present as of 3.8.1. Please feel free to give it a try and report back should you encounter any other issues.

cc: @chylex Please have a look at the fix and let me know if you have any questions or concerns.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants