Skip to content
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

Closed
oparoz opened this issue Feb 26, 2015 · 10 comments · Fixed by #16810
Closed

Nothing happens when users use the search field in most apps #14544

oparoz opened this issue Feb 26, 2015 · 10 comments · Fixed by #16810

Comments

@oparoz
Copy link
Contributor

oparoz commented Feb 26, 2015

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

@karlitschek
Copy link
Contributor

@butonic What do you think?

@nickvergessen nickvergessen added this to the 8.1-current milestone Mar 2, 2015
@karlitschek
Copy link
Contributor

@jancborchardt Isn´t this the bug that we discussed friday?

@DeepDiver1975
Copy link
Member

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.

@DeepDiver1975
Copy link
Member

@jancborchardt maybe find the time to fix this these days ....

@DeepDiver1975 DeepDiver1975 self-assigned this May 5, 2015
@jancborchardt
Copy link
Member

@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.

@jancborchardt
Copy link
Member

@DeepDiver1975 this we also need to work on tomorrow when coworking.

@DeepDiver1975
Copy link
Member

This is a tough one ....

  • how to detect that the current app is handling search results by itself?
  • how should the fallback look like?

@nickvergessen
Copy link
Contributor

how to detect that the current app is handling search results by itself?

I fixed this part in my previous attempt: https://github.com/owncloud/core/pull/16480/files

@oparoz
Copy link
Contributor Author

oparoz commented Jun 8, 2015

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.
I'm for introducing a way to disable the search field if the app hasn't registered to use it, so that each app doesn't have to create their own search field.

Having the old dropdown would prolly confuse users as they would be redirected to the Files app when selecting a file or folder.

@jancborchardt
Copy link
Member

Good call by @DeepDiver1975 to just hide it when the app doesn’t support search. Please review #16810

@lock lock bot locked as resolved and limited conversation to collaborators Aug 11, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants