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.

Sign up for GitHub

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

Jump to bottom

Fixing a search error #655

Closed
wants to merge 1 commit into from
Closed

Fixing a search error #655

wants to merge 1 commit into from

Conversation

tohidemyname
Copy link

searchText retruns wrong index.
@asfgit asfgit closed this in 894ef6e Jul 7, 2024
@centic9
Copy link
Member

centic9 commented Jul 7, 2024

Thanks for the suggestion, applied the actual fix now.

Note: I did not change to getRList() as this can have negative impact on performance, we saw that calling getRList() may introduce additional overhead.

Also did not add the appendText() method, please add a separate PR with accompanying tests if you would like to add such a method.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
2 participants
@tohidemyname @centic9