Replace "return" with "continue" in scroll_with_cursor #80
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
I was looking at the repo for inspiration and seemed to have found an apparent logical error: Throughout the
scroll_with_cursor
function,continue
is used to ensure other text inputs in the query are handled even if one is skipped. This appears to not be true for one part of the code, where itreturn
s instead.I figured I would PR the presumed fix instead of just opening an issue, however I did NOT test this change, as I'm not a user of the code at this time. Hope this still helps.