Conversation
c571a69 to
92659d1
Compare
|
Just curious on the use case for combining results. Is it just a personal preference? The reordering is a good idea. I don't know why I put collections second. |
Pretty much. I've noticed that some streaming services split results into categories, and some mix them all together. Plex mixes them all together by default but has a tab to switch to the category view. For me personally I prefer combined because when I'm searching I generally already know what I'm looking for, my search is going to closely match the title, so I prefer it to be the first result regardless of what category it falls under, if that makes sense. |
|
That makes sense. What do you think about adding a "View Options" button (like on library pages) on the search page and having the toggle there instead? Maybe to the right of the search text box? I'm trying to keep the settings pages from being an endless list of toggles. For the combined results, I think putting them in a |
app/src/main/java/com/github/damontecres/wholphin/ui/main/SearchPage.kt
Outdated
Show resolved
Hide resolved
Yeah, both of those thoughts make sense to me. I remember you're pretty wary of adding more toggles to the settings menu, haha. |
d020078 to
5cee096
Compare
|
@damontecres Just finished up with the changes based on your review. I moved the toggle from the settings page to the search page, applied the relevance scoring to both search options, and switched the combined results to show using cardgrid.
|




Description
Improves how search results are displayed:
Screenshots
AI/LLM usage
Code auditing and PR description drafting with Claude Code. Also used Claude Code for the regex.