From 2c9dd18f1b4d5697adcd5bdbfd094f7c9557064b Mon Sep 17 00:00:00 2001 From: Ben McCann <322311+benmccann@users.noreply.github.com> Date: Thu, 22 Feb 2024 09:58:48 -0800 Subject: [PATCH 01/28] fix: upgrade SvelteKit to 2.5.1 (#7351) --- web/package-lock.json | 8 ++++---- web/package.json | 2 +- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/web/package-lock.json b/web/package-lock.json index 84bd64d3e..78e5caf7c 100644 --- a/web/package-lock.json +++ b/web/package-lock.json @@ -32,7 +32,7 @@ "@socket.io/component-emitter": "^3.1.0", "@sveltejs/adapter-static": "^3.0.1", "@sveltejs/enhanced-img": "^0.1.8", - "@sveltejs/kit": "^2.5.0", + "@sveltejs/kit": "^2.5.1", "@sveltejs/vite-plugin-svelte": "^3.0.2", "@testing-library/jest-dom": "^6.1.5", "@testing-library/svelte": "^4.0.3", @@ -1859,9 +1859,9 @@ } }, "node_modules/@sveltejs/kit": { - "version": "2.5.0", - "resolved": "https://registry.npmjs.org/@sveltejs/kit/-/kit-2.5.0.tgz", - "integrity": "sha512-1uyXvzC2Lu1FZa30T4y5jUAC21R309ZMRG0TPt+PPPbNUoDpy8zSmSNVWYaBWxYDqLGQ5oPNWvjvvF2IjJ1jmA==", + "version": "2.5.1", + "resolved": "https://registry.npmjs.org/@sveltejs/kit/-/kit-2.5.1.tgz", + "integrity": "sha512-TKj08o3mJCoQNLTdRdGkHPePTCPUGTgkew65RDqjVU3MtPVxljsofXQYfXndHfq0P7KoPRO/0/reF6HesU0Djw==", "dev": true, "hasInstallScript": true, "dependencies": { diff --git a/web/package.json b/web/package.json index 1542acc2d..2b53d0645 100644 --- a/web/package.json +++ b/web/package.json @@ -27,7 +27,7 @@ "@socket.io/component-emitter": "^3.1.0", "@sveltejs/adapter-static": "^3.0.1", "@sveltejs/enhanced-img": "^0.1.8", - "@sveltejs/kit": "^2.5.0", + "@sveltejs/kit": "^2.5.1", "@sveltejs/vite-plugin-svelte": "^3.0.2", "@testing-library/jest-dom": "^6.1.5", "@testing-library/svelte": "^4.0.3", From 6bb30291de0a41bcd941d9b8dc6bfe0c3f0feb00 Mon Sep 17 00:00:00 2001 From: Michel Heusschen <59014050+michelheusschen@users.noreply.github.com> Date: Thu, 22 Feb 2024 19:08:55 +0100 Subject: [PATCH 02/28] fix(web): consistent combobox style + improve color contrast (#7353) --- web/src/lib/components/shared-components/combobox.svelte | 4 ++-- .../components/user-settings-page/appearance-settings.svelte | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/web/src/lib/components/shared-components/combobox.svelte b/web/src/lib/components/shared-components/combobox.svelte index 45ea116e3..7c1e1f68d 100644 --- a/web/src/lib/components/shared-components/combobox.svelte +++ b/web/src/lib/components/shared-components/combobox.svelte @@ -52,7 +52,7 @@ }; -
+
{#if isOpen}
@@ -96,7 +96,7 @@
{#if filteredOptions.length === 0}
No results
diff --git a/web/src/lib/components/user-settings-page/appearance-settings.svelte b/web/src/lib/components/user-settings-page/appearance-settings.svelte index db6f28a36..0326cab41 100644 --- a/web/src/lib/components/user-settings-page/appearance-settings.svelte +++ b/web/src/lib/components/user-settings-page/appearance-settings.svelte @@ -80,7 +80,7 @@ checked={$locale == undefined} on:toggle={handleToggleLocaleBrowser} > -

{selectedDate}

+

{selectedDate}

{#if $locale !== undefined} From b3b6426695f222ebc220d307c53cfe8d6caccab4 Mon Sep 17 00:00:00 2001 From: martin <74269598+martabal@users.noreply.github.com> Date: Fri, 23 Feb 2024 06:01:19 +0100 Subject: [PATCH 03/28] feat(web): configure slideshow (#7219) * feat: configure slideshow delay * feat: show/hide progressbar * fix: slider * refactor: use grid instead of flex * fix: default delay * refactor: progress bar props * refactor: slideshow settings * fix: enforce min/max value * chore: linting --------- Co-authored-by: Jason Rasmussen --- .../machine-learning-settings.svelte | 10 +-- .../asset-viewer/slideshow-bar.svelte | 39 +++++----- web/src/lib/components/elements/slider.svelte | 72 +++++++++++++++++++ .../components/photos-page/asset-grid.svelte | 6 +- .../progress-bar/progress-bar.svelte | 45 ++++++------ .../settings/setting-input-field.svelte | 20 ++++-- .../settings/setting-select.svelte | 4 +- .../settings/setting-switch.svelte | 64 +---------------- .../lib/components/slideshow-settings.svelte | 37 ++++++++++ web/src/lib/stores/slideshow.store.ts | 5 ++ 10 files changed, 183 insertions(+), 119 deletions(-) create mode 100644 web/src/lib/components/elements/slider.svelte create mode 100644 web/src/lib/components/slideshow-settings.svelte diff --git a/web/src/lib/components/admin-page/settings/machine-learning-settings/machine-learning-settings.svelte b/web/src/lib/components/admin-page/settings/machine-learning-settings/machine-learning-settings.svelte index 6a542d81d..ba24f3aab 100644 --- a/web/src/lib/components/admin-page/settings/machine-learning-settings/machine-learning-settings.svelte +++ b/web/src/lib/components/admin-page/settings/machine-learning-settings/machine-learning-settings.svelte @@ -112,8 +112,8 @@ desc="Minimum confidence score for a face to be detected from 0-1. Lower values will detect more faces but may result in false positives." bind:value={config.machineLearning.facialRecognition.minScore} step="0.1" - min="0" - max="1" + min={0} + max={1} disabled={disabled || !config.machineLearning.enabled || !config.machineLearning.facialRecognition.enabled} isEdited={config.machineLearning.facialRecognition.minScore !== savedConfig.machineLearning.facialRecognition.minScore} @@ -125,8 +125,8 @@ desc="Maximum distance between two faces to be considered the same person, ranging from 0-2. Lowering this can prevent labeling two people as the same person, while raising it can prevent labeling the same person as two different people. Note that it is easier to merge two people than to split one person in two, so err on the side of a lower threshold when possible." bind:value={config.machineLearning.facialRecognition.maxDistance} step="0.1" - min="0" - max="2" + min={0} + max={2} disabled={disabled || !config.machineLearning.enabled || !config.machineLearning.facialRecognition.enabled} isEdited={config.machineLearning.facialRecognition.maxDistance !== savedConfig.machineLearning.facialRecognition.maxDistance} @@ -138,7 +138,7 @@ desc="The minimum number of recognized faces for a person to be created. Increasing this makes Facial Recognition more precise at the cost of increasing the chance that a face is not assigned to a person." bind:value={config.machineLearning.facialRecognition.minFaces} step="1" - min="1" + min={1} disabled={disabled || !config.machineLearning.enabled || !config.machineLearning.facialRecognition.enabled} isEdited={config.machineLearning.facialRecognition.minFaces !== savedConfig.machineLearning.facialRecognition.minFaces} diff --git a/web/src/lib/components/asset-viewer/slideshow-bar.svelte b/web/src/lib/components/asset-viewer/slideshow-bar.svelte index c099ae79c..164b0a791 100644 --- a/web/src/lib/components/asset-viewer/slideshow-bar.svelte +++ b/web/src/lib/components/asset-viewer/slideshow-bar.svelte @@ -1,23 +1,16 @@
- dispatch('close')} title="Exit Slideshow" /> - {#if $slideshowShuffle} - ($slideshowShuffle = false)} title="Shuffle" /> - {:else} - ($slideshowShuffle = true)} title="No shuffle" /> - {/if} + dispatch('close')} title="Exit Slideshow" /> (progressBarStatus === ProgressBarStatus.Paused ? progressBar.play() : progressBar.pause())} title={progressBarStatus === ProgressBarStatus.Paused ? 'Play' : 'Pause'} /> - dispatch('prev')} title="Previous" /> - dispatch('next')} title="Next" /> + dispatch('prev')} title="Previous" /> + dispatch('next')} title="Next" /> + (showSettings = !showSettings)} title="Next" />
+{#if showSettings} + (showSettings = false)} /> +{/if} +