-
-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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.
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
Use unified search input to filter in files #23432
Comments
Yeah, this would still be the expected experience. @skjnldsv are you aware of the regression? cc @tobiasKaminsky since he mentioned something similar to me as well. |
Yeah, we decided we were ok with this in one of the calls. |
I dare to disagree…
|
Not saying what is better or not, saying that was intended as discussed in the calls. |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Maybe the search result list could split the results in two parts: the local search results of the current app and the global ones. If the unified search field would at the same time filter the file list it would feel strange as they are visually disconnected somehow. It would rather require another search field specifically on top of the file list, but seeing two fields might feel redundant. Or hide it behind a "filter files" icon with a dropdown showing the box. Having it closer to the file list would make a visual connection. |
Per feedback from users, the search result list is felt to be awfully small. This usability aspect was covered in #24029, and shouldn't get lost. |
@jancborchardt I was looking into reintroducing the local folder filtering however there is a quite big UX issue in my oppinion as the search input is now hidden in the popover and once that is closed there it is not really clear how to cancel the filtering or what query is currently active: Compared to the old version where the input was always visible: Any good idea how we could solve that? |
Similar is already in /settings/apps, where one can filter in popup also current app list, closes popup, filtering remains and one has to click on search to cancel within popup. @skjnldsv symbol next to deck then should have a cancel, or? like: |
I like your solution @tobiasKaminsky |
At first I thought it would be very easy to implement, until I realised File's unified search provider has no ways to know from which folder the search was triggered 😢 |
That would be cool |
It seems to me that the UI part of this issue is related to #27078 |
seems this PR brings it back: #26841 |
Until NC 19 it was possible to filter the shown files and folders in the files app by the search string. With the unified search, this is no longer possible which makes it cumbersome to find a certain file or folder in a folder with many entries, especially since entries are loaded paginated.
The files app should just as well listen for the event emitted by the unified search and filter accordingly.
The text was updated successfully, but these errors were encountered: