From 14825e4daf8c782eb8d72cfafae5bf3a2d813ff0 Mon Sep 17 00:00:00 2001
From: Jason Zimdars <jz@37signals.com>
Date: Wed, 30 Apr 2025 10:53:08 +0200
Subject: [PATCH] Don't trigger other hotkeys when dismissing

---
 app/views/layouts/_lightbox.html.erb | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/app/views/layouts/_lightbox.html.erb b/app/views/layouts/_lightbox.html.erb
index 1f5ad9105..91f3af7bc 100644
--- a/app/views/layouts/_lightbox.html.erb
+++ b/app/views/layouts/_lightbox.html.erb
@@ -1,10 +1,10 @@
 <dialog class="lightbox" aria-label="Image Viewer (Press escape to close)" data-lightbox-target="dialog" data-action="close->lightbox#reset">
   <img src="" class="lightbox__image" data-lightbox-target="zoomedImage" />
 
-  <form method="dialog" class="lightbox__btn">
-    <button class="btn fill-white" title="Close (esc)">
+  <div class="lightbox__btn">
+    <button class="btn fill-white" title="Close (esc)" data-action="keydown.esc->lightbox#close:stop" data-controller="hotkey">
       <%= icon_tag "remove" %>
       <span class="for-screen-reader">Close image viewer (esc)</span>
     </button>
-  </form>
+  </div>
 </dialog>