[web-app-search] No visible difference between folders and spaces in search results

[hurradieweltgehtunter]

Steps to reproduce

1. Make sure to have min 1 folder and a space created with the same name
2. Search this name in global search

Expected behaviour

Saerch results are somehow differentiated based on their type (folder, space, ...)

Actual behaviour

No difference visible

First search result in screenshot (top list and full view list) is a space, other are folders (see edit pen, rename possible for folders, not for spaces)

Also, I expected that the dropdown result list would segment results based in their type (see top left "Files")

https://ocis.ocis-wopi.latest.owncloud.works

[hurradieweltgehtunter]

sth like this would be nice

[kulmann]

Doable by splitting the search provider in the files app into two different providers, one filtering out spaces from the result set, the other one filtering everything but spaces out. This would be reasonable effort for the search result preview. But we should consider the full picture here - hitting Enter the user lands on a search result page. That should take the same thoughts into account.

There are some more more or less related issues:

• Improve Search Experience #5844
• Search result page is missing basic features #7780

And there is also the idea that the search preview does even more segmentations:

• (FR) Change behaviour of Search Bar to list results from current folder listing first #7099

Nice enhancement, but needs more concepting. :-)

[tbsbdr]

I'd stick to the scheme and show the following, if the result is a Space:

1. display "Spaces" as parent folder
2. show Space Image/Icon if

[JammingBen]

I stumbled upon this while fixing #9022.

Doable by splitting the search provider in the files app into two different providers, one filtering out spaces from the result set, the other one filtering everything but spaces out.

@kulmann Do we really need separate search providers for this? My current fix (#9026) simply adjusts the existing search provider and returns the correct space resource if found.

[kulmann]

I stumbled upon this while fixing #9022.

Doable by splitting the search provider in the files app into two different providers, one filtering out spaces from the result set, the other one filtering everything but spaces out.

@kulmann Do we really need separate search providers for this? My current fix (#9026) simply adjusts the existing search provider and returns the correct space resource if found.

Your solution is good. 👍

[JammingBen]

Fixed via #9026.