Fix and improve get_status_text display in prompt
#104
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.
Looking at how the
get_status_textfunction was rendering in the prompt, it was counting#is_addedrecords. This must have broke when the table switched to tracking non-numeric indices at some point (I assume). This PR fixes that, and also improves it to showshowing_results / total_indexedstyle output, much like Telescope's stock finders...Before

After

The first
showing_resultsnumber will always be limited to how many results are actually showing (limited by yourresult_limitconfig), but I think this explicit clarity will actually be helpful for people who arrow through results. If the number shows less than the total, it's clear that there's more results found, but their prompt may not be specific enough.I've also fixed the ability for users to override
get_status_textvia opts. This looked like it was possible, but never worked. For example...