Close edit modal when overlay is clicked

Remove unnecessary dataset action check in modal close method. Update
modal overlay to use click event for closing, ensuring consistent modal
dismissal behavior.

Author: fbuys

app/javascript/controllers/modal_controller.js

@@ -4,8 +4,6 @@ export default class extends Controller {
   static targets = ["content"]
 
   close(event) {
-    if (event.target.dataset.action === "modal#close") {
-      this.element.remove()
-    }
+    this.element.remove()
   }
 }

app/views/puzzles/edit.html.erb

@@ -1,6 +1,5 @@
-
 <turbo-frame id="modal">
-  <div data-controller="modal" data-action="modal#close"class="modal-overlay">
+  <div data-controller="modal" data-action="click->modal#close"class="modal-overlay">
     <div data-modal-target="content"
         class="modal-content"
         onclick="event.stopPropagation()">