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

GetLinesInViewport should consider ignore unchanged. #123

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

Conversation

kbeatty-ascensionas
Copy link
Contributor

No description provided.

@kbeatty-ascensionas
Copy link
Contributor Author

Closes #122

@mmanela
Copy link
Owner

mmanela commented Jun 10, 2024

@kingcean Could you review?

@mmanela
Copy link
Owner

mmanela commented Jul 13, 2024

@kbeatty-ascensionas Can you share an example text to test with to see the behavior so I can validate?

@kbeatty-ascensionas
Copy link
Contributor Author

@kbeatty-ascensionas
Copy link
Contributor Author

What I was seeing is the early breakout in the loop would happen as soon as the first not in the viewport line was encountered. This was exposed by my previous fix that marked zero height lines as NOT being in the view.

Ignore the image, it illustrates my previous fix. Not this one.

@kbeatty-ascensionas
Copy link
Contributor Author

image
This image illustrates the new problem that I see.

@kbeatty-ascensionas
Copy link
Contributor Author

@mmanela do you have any comments on this?

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