feat(core): redesign unified search result presentation - #62605
Conversation
|
@susnux thanks for adding labels, but I am going to remove the |
85a050a to
cc6f951
Compare
de55ed6 to
eb473e4
Compare
|
Thanks for the feedback @kra-mo , I will fix the minor UI issues.
Ordering is set per provider, so we can tune it as needed. And yes, Apps is deliberately pinned above everything since it's the launcher, and below that the current app's provider should lead. That mostly holds, but the values are inconsistent (user management currently ranks itself near the bottom on its own page), so I'll audit the core providers and normalize them. Longer term we could also bucket the providers server-side, so the current app's block always comes right after Apps even when the app's own provider doesn't ask for it. There isn't always a clean mapping between the app in the route and the provider's app ID though, so in those cases we'd fall back to the order the provider specifies. Either way, this can happen iteratively :)
You mean in the sense that it does not open the filter dropdown at all? I tested on both FF and Chromium, across various apps, and it always opens for me. Do you see any errors in the console? |
Yeah, that is nice.
Yes, in Safari, no errors that I can see. |
@kra-mo @pringelmann yeah, sorry I said only to show it on hover, but if it’s shown on both hover and focus it would be fine? Then 1 time you have to tab into it as a keyboard user, but then you know the shortcut. |
|
But focus makes you be able to type directly. In any case, I will explore options for making it less distracting. |
|
@kra-mo I fixed the minor UI issues as well as the safari bug (will make sure to test in Safari in future before opening PRs) |
kra-mo
left a comment
There was a problem hiding this comment.
- We have variables for clickable elements and headings now. They should be used:
--font-weight-elementand--font-weight-heading. - The bar doesn't seem to appear for me properly and the Apps entry still has a dark foreground (although don't spend too much time fixing it because of #62728).
Screen.Recording.2026-08-03.at.16.54.38.mov
Looks nice otherwise :)
4849324 to
ee1044d
Compare
|
@kra-mo I've fixed the fonts and bar rendering on safari. The incorrect app icon color is not related to the PR, but rather my preview env setup. (I will fix this at some point) |
kra-mo
left a comment
There was a problem hiding this comment.
Sorry, this one is still not --font-weight-element:
Once that's fixed, consider this approved :)
And another thing I just happened to notice but don't take it as blocking this PR is that on mobile, the text for the filters can start truncating early enough so the icons should probably either be hidden at a certain breakpoint, or the alignment should change depending on size:
|
/compile rebase |
Signed-off-by: Peter Ringelmann <peter.ringelmann@nextcloud.com>
Signed-off-by: Peter Ringelmann <peter.ringelmann@nextcloud.com>
Signed-off-by: Peter Ringelmann <peter.ringelmann@nextcloud.com>
Signed-off-by: Peter Ringelmann <peter.ringelmann@nextcloud.com>
Signed-off-by: Peter Ringelmann <peter.ringelmann@nextcloud.com>
Signed-off-by: Peter Ringelmann <peter.ringelmann@nextcloud.com>
Signed-off-by: Peter Ringelmann <peter.ringelmann@nextcloud.com>
Signed-off-by: Peter Ringelmann <peter.ringelmann@nextcloud.com>
Signed-off-by: Peter Ringelmann <peter.ringelmann@nextcloud.com>
Signed-off-by: Peter Ringelmann <peter.ringelmann@nextcloud.com>
1f53e82 to
7808932
Compare

ctrl+fa second time does not allow native search #62626Summary
Reworks how unified search presents results, per the design in #61652.
The aggregate list now shows at most three rows per category, each group headed by a "More from …" control that opens that category's full result page in a detail view. Result rows show their provider's real icon instead of a generic one.
-> Preview link <- (u:
bob, p:bob)Smaller fixes from the same pass:
Design feedback already captured
The following feedback items were already captured. They will land in a follow-up PR: I want to get the core work merged, then we iterate.
Also see screenshot: the PDF filetype icon is too small and also blue
Checklist
3. to review, feature component)stable32)AI (if applicable)