From 1765d70911d392738be21120f027c5c4f42f8011 Mon Sep 17 00:00:00 2001
From: Sli <antoine@bartuccio.fr>
Date: Wed, 19 Aug 2026 16:09:11 +0200
Subject: [PATCH] Move image click to frame and remove image selection

---
 sas/static/sas/css/picture.scss | 3 +++
 sas/templates/sas/picture.jinja | 7 +++++--
 2 files changed, 8 insertions(+), 2 deletions(-)

diff --git a/sas/static/sas/css/picture.scss b/sas/static/sas/css/picture.scss
index 4ed49eea..d22a4441 100644
--- a/sas/static/sas/css/picture.scss
+++ b/sas/static/sas/css/picture.scss
@@ -48,6 +48,9 @@
       height: 100%;
       max-width: 100%;
       object-fit: contain;
+      // Prevent ugly selection when
+      // clicking to change the image
+      user-select: none;
     }
   }
 }
diff --git a/sas/templates/sas/picture.jinja b/sas/templates/sas/picture.jinja
index b93e6a89..ebc7a53c 100644
--- a/sas/templates/sas/picture.jinja
+++ b/sas/templates/sas/picture.jinja
@@ -82,13 +82,16 @@
 
     <div class="container" id="pict">
       <div class="main">
-        <div class="photo" :aria-busy="currentPicture.imageLoading">
+        <div
+          class="photo"
+          :aria-busy="currentPicture.imageLoading"
+          @click="currentPicture = (($event.offsetX > $event.target.getBoundingClientRect().width / 2) ? nextPicture : previousPicture) ?? currentPicture"
+        >
           <img
             :src="currentPicture.compressedUrl"
             :alt="currentPicture.name"
             id="main-picture"
             x-ref="mainPicture"
-            @click="currentPicture = (($event.offsetX > $el.offsetWidth / 2) ? nextPicture : previousPicture) ?? currentPicture"
           />
         </div>