From b3b258f32f0a08425466d6dff75a19bfef24fbd8 Mon Sep 17 00:00:00 2001 From: Lukas Date: Fri, 24 May 2024 10:56:36 +0200 Subject: [PATCH] fix(web): allow copying text in photo viewer (#9705) * fix(web): allow copying text in photo viewer * use default browser copy * revert changes * fix lint --- web/src/lib/components/asset-viewer/photo-viewer.svelte | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/web/src/lib/components/asset-viewer/photo-viewer.svelte b/web/src/lib/components/asset-viewer/photo-viewer.svelte index f541c1ca0a..7ad0cd51fe 100644 --- a/web/src/lib/components/asset-viewer/photo-viewer.svelte +++ b/web/src/lib/components/asset-viewer/photo-viewer.svelte @@ -120,10 +120,11 @@ forceLoadOriginal = state.currentZoom > 1 && isWebCompatibleImage(asset) ? true : false; }); - const onCopyShortcut = () => { + const onCopyShortcut = (event: KeyboardEvent) => { if (window.getSelection()?.type === 'Range') { return; } + event.preventDefault(); handlePromiseError(doCopy()); }; @@ -132,8 +133,8 @@ on:copyImage={doCopy} on:zoomImage={doZoomImage} use:shortcuts={[ - { shortcut: { key: 'c', ctrl: true }, onShortcut: onCopyShortcut }, - { shortcut: { key: 'c', meta: true }, onShortcut: onCopyShortcut }, + { shortcut: { key: 'c', ctrl: true }, onShortcut: onCopyShortcut, preventDefault: false }, + { shortcut: { key: 'c', meta: true }, onShortcut: onCopyShortcut, preventDefault: false }, ]} /> {#if imageError}