Unified search: followups

[skjnldsv]

Providers

• Implement Contacts Backend for Unified Search #22011
• Add Event and Task Backends for Unified Search #22020 (@georgehrke)
• Add settings sections to unified search #22095
• Enabled apps App navigation via unified search #22102

Needed

Enhancements

• Check the additional filters feature, interface for the "in:files" etc Allow unified search filtering and bump @nextcloud/vue #22111
• Add global event Emit unified search query #22526
• Users & apps filtering

Issues

• Sort results of currently active app first Make apps handle the order logic #22104
• The search icon should remain white even in dark mode
• Click load more on paginated type show the loading emptycontent
• Files are lib/composer/composernot showed with appropriate mime icons
• When you set the theming color to #FFFFFF and #181818 respectively the section headers are invisible on bright/dark theme.
• A problem might be that the search is not persistent, as soon as the unified search input is closed, the filtering is reset Emit unified search query #22526 (comment)

Unrelated

• Ctrl-F doesn’t work with Text enabled (@juliushaertl) Only overwrite Ctrl-f when text is focussed text#988

Documentation

• How to implement inner-app search (eventListener prevent default)
• Update php API requirements
• Simple example

Order

• [-50] App search itself
• [-1] Current app
• [5] Files
• [10] Comments
• [15] Talk @nickvergessen
• [20] Mail @ChristophWurst
• [25] Contacts
• [30] Calendar
• [35] Tasks
• [500] Settings
• Install apps - Not implemented
• Documentation - Not implemented

Future Improvements

• SHow placeholder instead of loader
• Suggestions as emptycontent e.g. pre-filters
• See how unified search relates to the Contacts menu (depending on how we improve the Contacts menu)
• Alignment further to the right
• In the case of Files, for a file or folder which is in the current folder we don’t really need to refresh for the scrollto highlight – possible enhancement?
• Define criteria for relevant results and sort them first

cc @jancborchardt

[jancborchardt]

When you set the theming color to #FFFFFF and #181818 respectively the section headers are invisible on bright/dark theme.

Can be fixed by using var(--color-primary-element) instead of just color-primary for the headers. :)

[georgehrke]

Apps ?
Users ?

@skjnldsv @nickvergessen This actually has a similar problem like Contacts Backend:
https://github.com/nextcloud/server/pull/22011/files#diff-7906ed6e5e203f2a8cdfd4849bc52ba2R99

At the time of registration in boot(), an app doesn't know whether the current user is admin / subadmin.
What i realized after commenting on the Contacts thing:
Since it just returns an empty list, it won't show a Contacts section in the result list, but the placeholder of the search field will still include "Search for ..., Contacts, ...".

Should we extend OCP\Search\IProvider with an isAvailableForUser(IUser $user) method?

[nickvergessen]

We could add a function whether the search provider is applicable which is then invoked/checked when necessary.

[skjnldsv]

I think this is a very desirable thing to have @skjnldsv. Every platform that has global search and apps has this feature.
On top of that, both on mobile and desktop operating systems the global search is becoming the main way people navigate and launch apps.

Jan said yes! 🚀 So let's add this 🎉 😉

[marcoambrosini]

...and I would also include the search for apps that are not installed, with a redirection to the app store, much like:

(the first search is an installed app, the second one is a not installed app)

[marcoambrosini]

On a different note I think that in the future it would be nice to have more space for the results, so that we can include more actions and previews. Like putting it in a big dialog or do a complete UI overlay, similar to what respectively MacOS and Gnome do,

[jancborchardt]

Yep, @skjnldsv and I talked about this – the idea is indeed to have a universal search like Spotlight, GNOME search, iOS search etc. :) Including current apps, non installed apps, settings etc. :)

[skjnldsv]

Hey! :)
No problems!

I have people asking me to implement it in my applications.

Fulltextsearch or the unified search?

[jancborchardt]

I don’t know about the technical underpinnings, but UX wise of course the new search should also be able to do full text search inside of documents.

I assume it’s mostly a question of the fulltextsearch app hooking in to the new unified search API?

[matiasdelellis]

You're right.
I guess that the correct proposal should be that fulltextsearch will be integrated as an provider of the unified search.

[raimund-schluessler]

Since I don't really know where to ask this, or if it needs a separate issue, I just ask here:

What is the planned way to allow apps to filter their content. For the Tasks app, I used the searchBox to allow a user to filter the tasks. The shown tasks would immediately update.
With the searchBox gone, what is the alternative now?

[ChristophWurst]

For filtering we suggest an in-app solution, like you find in Talk right now.

[skjnldsv]

• No results flashes briefly before a search
• Do not show apps results if you're on /apps/settings/apps
• Add loading spinners on unfinished search providers requests
• Cancel requests on close
• When the app does inline filtering and you click somewhere else and the popover closes, the popover doesn’t fully close but the search field still shows?