-
-
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
feat: In app search #41609
feat: In app search #41609
Conversation
0637db7
to
0291b2a
Compare
I really like this idea! What is needed to finish this? |
@szaimen we'll have to address proper in app search strategy this for the next major. @jancborchardt had usability concerns about this solution. I think it's a pretty solid starting point compared to other alternatives, but of course I'm happy to iterate over this or explore other solutions and see how they compare. |
What kind of usability concerns?
Indeed, I think it is much better in terms of usability than the current situation. |
Closing as there's no need for a last minute solution anymore, and it seems that we all favor proper in app search that's tailored to the respective apps and thus slightly different in each case. |
@marcoambrosini I still think this could be a good combination of in-app and global search. It would also move us closer to the usual "big search field in the top center" that people know from Google and Microsoft et al. This was also a big point why you favored this approach, no @marcoambrosini? Also cc @AndyScherzinger since we recently talked about search. |
I agree 👍 |
I just tested this branch locally and it doesnt seem to work... @marcoambrosini would you mind updating it? :) |
0291b2a
to
7ab9adc
Compare
7ab9adc
to
e17729f
Compare
@szaimen there were some "heavy" adjustments needed to bring this API wise to a consistent state (not 100% related to this feature but we basically needed to split out the event handling to the main view and only handle the unified global search in the modal). I tested it locally and it seems to work as expected, some styles are broken due to the clickable area refactoring, but will be fixed soon I think with new nextcloud-vue version. Bildschirmaufnahme_20240625_211909.webm |
e17729f
to
efa3080
Compare
This comment was marked as resolved.
This comment was marked as resolved.
Could the input be always shown and centered if there's enough space? |
We would need to remove local search from those apps if we go with this imo |
Yes it works for all apps that implement the already used "filter in current view" (yet we need proper API ...) |
f2fd1e6
to
f2793a4
Compare
7af4fee
to
b0c8b50
Compare
b0c8b50
to
9161073
Compare
Co-authored-by: Ferdinand Thiessen <opensource@fthiessen.de> Signed-off-by: Marco Ambrosini <marcoambrosini@icloud.com>
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
Co-authored-by: Simon L. <szaimen@e.mail.de> Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
…h:reset` event Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
9161073
to
3294037
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@susnux @szaimen just before you put more time into this – we talked about this again among @AndyScherzinger @karlitschek and me, and in addition to the considerations in the previous comment we think that in the long term it would be better to have 2 searches:
- One present search field inside the app, like it is in Talk, Contacts, Mail, Collectives, etc.
- One search icon on the top right for the Unified search
And to do that, we would "simply" need to add in-app searches in:
- Files: As per mockups in Filetype filters on top of the files list (doc, pdf, jpg, ...) #45437
- Apps management: Top of the left navigation
- Account management: Top of the left navigation
This would also resolve the confusion with the bit strange interaction pattern of having an input field with a button that does something else, and with internal search not existing in specific apps and due to that the flow being different.
This is very comparable to how for example macOS or GNOME does it with every app having in-app search, and there being a global search (like Spotlight) in case you don’t find it.
@szaimen @susnux sorry for the back and forth – it just turns out it is way more complex than thought. :\
Hi @susnux this is not working on daily.ltd2.nextcloud.com anymore. Was it removed? |
Summary
This is an attempt to reconcile app search and global search into a single component and user flow.
Advantages of this approach:
Screencast.from.2023-11-20.17-13-37.webm
On mobile
TODO