The searchTerm in the searchPredicate function is never an empty string about react-dropdown-tree-select HOT 7 CLOSED

Is an empty string a valid search term? What use case do you anticipate for this?

I made a specific change that was accepted a few months ago to do this because there did not appear to be a good reason to incur the search predicate cost across a potentially large number of nodes with an empty string. See #580

from react-dropdown-tree-select.

In my project, we use the searchPredicate function to build a custom async tree dropdown. Now our component is not working as expected when the input is cleared, we should be able to show all the options in that case but we can't.

from react-dropdown-tree-select.

In my project, we use the searchPredicate function to build a custom async tree dropdown. Now our component is not working as expected when the input is cleared, we should be able to show all the options in that case but we can't.

So the change I had made would not filter anything if search mode was off, which it was if the query was an empty string. That is, I would expect the default behavior with an empty string is to show all options.

Is this something you can create a code sandbox demonstrating?

from react-dropdown-tree-select.

Hi, I have a sandbox here with an example: https://codesandbox.io/s/blue-snowflake-jqt35v?file=/src/index.js

if the search term is empty, it is not reflected that way in the searchPredicate

from react-dropdown-tree-select.

searchPredicate was meant to return matches, so an empty string doesn't mean it matches all. That can be a business interpretation for a particular use case but baking that in, makes the control opinionated.

I like the idea of a separate function to notify when the search input is empty but I wonder why there are so many changed files. We need to verify if every snapshot is still the same, which at a glance appears to be so but again, I don't like the fact that avajs decided to change the render sequence throwing the git diff off.

from react-dropdown-tree-select.

Thanks for your response @mrchief. The many file changes are indeed related to the avajs. I only executed ava --update-snapshots so no idea what went wrong on that regard.

from react-dropdown-tree-select.

This issue has been automatically marked as stale because it has not had recent activity. It will be closed in 10 days if no further activity occurs. Thank you for your contributions.

from react-dropdown-tree-select.