beenull/immich
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

fix(web): Add m: to search query upon loading results. (#2954)

Browse source 
Previously, we'd drop the m: from non-clip searches entirely. This
behavior incorrectly represents the page's status (results from
non-clip search but query implies a clip search). Also, any follow-up
searches change to clip searches, which feels like a jarring UX if you
have to add m: every time in a 'search-session'.
This commit is contained in:
Rohitt Vashishtha 2023-06-28 23:18:53 +05:30 committed by GitHub
parent 017214fd56
commit b4e641548c
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 8 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
web/src/routes/(user)/search/+page.svelte
View file

@ -43,7 +43,14 @@
}
});
$: term = $page.url.searchParams.get('q') || data.term || '';
$: term = (() => {
let term = $page.url.searchParams.get('q') || data.term || '';
const isMetadataSearch = $page.url.searchParams.get('clip') === 'false';
if (isMetadataSearch && term !== '') {
term = `m:${term}`;
}
return term;
})();
let selectedAssets: Set<AssetResponseDto> = new Set();
$: isMultiSelectionMode = selectedAssets.size > 0;