-
Notifications
You must be signed in to change notification settings - Fork 2.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
Nothing happens when users use the search field in most apps #14544
Comments
@butonic What do you think? |
@jancborchardt Isn´t this the bug that we discussed friday? |
We should introduce some fallback at least - I'm also a bit "mad" about the fact that in case I want to use the search box as a pure client side filter mechanism that there is still a request sent to the server to perform a search. |
@jancborchardt maybe find the time to fix this these days .... |
@DeepDiver1975 yeah, probably it’s best for us to collaborate on it when coworking. The important thing we need to decide is what to do for an app / view which does not support search – which will be most apps actually. If we want the search to have a purpose then we need to make it search the Files. In that case we need something similar to the previous search dropdown. But a bit wider, and nicely styled. |
@DeepDiver1975 this we also need to work on tomorrow when coworking. |
This is a tough one ....
|
I fixed this part in my previous attempt: https://github.com/owncloud/core/pull/16480/files |
Having given it some thoughts, I think we should respect the context. If we're in an app, the search field should work for data which the app can work with and that's the app's responsibility to implement that. Having the old dropdown would prolly confuse users as they would be redirected to the Files app when selecting a file or folder. |
…ent selected app - fixes #14544
Good call by @DeepDiver1975 to just hide it when the app doesn’t support search. Please review #16810 |
As mentioned here, here and here, the search field is broken for most apps because the mechanism has changed in oC8 and there is no fallback in place.
The fallback could be for the field to work like in previous versions, giving users access to a dropdown or a more drastic change could be implemented by removing the field if the app hasn't registered with the search engine.
To me, this is a regression, but I'll let the people in charge be judge of that.
@butonic @jancborchardt @DeepDiver1975
The text was updated successfully, but these errors were encountered: