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

fix: Correct taking content after cursor #88

Closed
wants to merge 1 commit into from

Conversation

Palm1r
Copy link
Owner

@Palm1r Palm1r commented Mar 4, 2025

This fix solves the problem when the text after the cursor from the same line was not included in the query

@Palm1r
Copy link
Owner Author

Palm1r commented Mar 4, 2025

@p12tic Sorry, this problem has been bothering me, is this fix okay for you?

@p12tic
Copy link
Contributor

p12tic commented Mar 4, 2025

Perhaps worth merging after #90 so that it's clear how behavior is changed.

@p12tic
Copy link
Contributor

p12tic commented Mar 5, 2025

If you don't mind I can update tests for this PR as well

@Palm1r Palm1r force-pushed the fix-getting-content-after-cursor branch from 3bbbb2c to 5aaf3db Compare March 5, 2025 14:12
@p12tic
Copy link
Contributor

p12tic commented Mar 5, 2025

I looked into test failures and it seems this needs more work.

@Palm1r
Copy link
Owner Author

Palm1r commented Mar 5, 2025

@p12tic yeah, you can continue the PR

@Palm1r Palm1r closed this Mar 5, 2025
@Palm1r Palm1r deleted the fix-getting-content-after-cursor branch March 8, 2025 12:26
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