From 17e59de59c5f6dbba3e7e142bd521cc366b7f413 Mon Sep 17 00:00:00 2001 From: Manav Rathi Date: Wed, 12 Feb 2025 15:11:42 +0530 Subject: [PATCH] Revert "Note diversion that didn't work" This reverts commit 0791a8f659e7aad48ff69c53f07c7894db663345. --- .../new/photos/components/FileViewerPhotoSwipe.ts | 10 ++-------- 1 file changed, 2 insertions(+), 8 deletions(-) diff --git a/web/packages/new/photos/components/FileViewerPhotoSwipe.ts b/web/packages/new/photos/components/FileViewerPhotoSwipe.ts index b39dfdbe51..b3363b04ea 100644 --- a/web/packages/new/photos/components/FileViewerPhotoSwipe.ts +++ b/web/packages/new/photos/components/FileViewerPhotoSwipe.ts @@ -161,14 +161,8 @@ export class FileViewerPhotoSwipe { // The default imageClickAction is "zoom-or-close". When the image // is small and cannot be zoomed into further (which is common when // just the thumbnail has been loaded), this causes PhotoSwipe to - // close. - // - // This can be disabled by setting `clickToCloseNonZoomable` to - // false, but we also want to disable zooming into the thumbnails - // (it's not useful, and further, the user's zoom will be lost when - // the full res is loaded). Setting `imageClickAction` to "zoom" - // achieves both these ends. - imageClickAction: "zoom", + // close. Disable this behaviour. + clickToCloseNonZoomable: false, // The default `bgClickAction` is "close", but it is not always // apparent where the background is and where the controls are, // since everything is black, and so accidentally closing PhotoSwipe