load all necessary records in searchNext request

[wnleao]

Could you take a look at #49 and let me know what you think?

I was working testing some things in the frontend and I thought that loading all necessary records and sending them back in the same search response would lead to a better flow when searching for terms. This way we don't see the flickering (with loading progress) in the screen anymore and we get the next occurrence index in the search response as well.

[sevdokimov]

Thanks. Will review today/tomorrow.

[sevdokimov]

Hi!
Thank you for the PR. Search looks much better when no flickering.
I merged your PR into search_next_load_all branch and slightly modified it. Please, look.

[wnleao]

Hey! Sure, I'm gonna take a look.

[wnleao]

Great changes, thank you! The only question that I have is related to the evaluation of loadNext after a search is performed. If loadNext is false, but there is some data available, shouldn't you compute and return the corresponding "foundIdx", either 0 or data.length-1? In the current scenario -1 will be returned.

[sevdokimov]

yes, you are right. Fixed.

[wnleao]

nice! I've created the PR.

[sevdokimov]

Done. The fix is merged to "master" branch.