From 0fc4475040fe6f2be5d04d7f52702120b451c39e Mon Sep 17 00:00:00 2001
From: Ryan Chen <ryan@torrtle.co>
Date: Wed, 26 Nov 2025 20:59:23 -0500
Subject: [PATCH] Fix delete button functionality and remove confirmation
 dialog
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

- Fix JavaScript event parameter passing in channels page
- Add event parameter to refreshChannel and deleteChannel functions
- Update onclick handlers to pass event object
- Remove confirmation dialog from channel deletion
- Remove success alert, just reload page after deletion

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
---
 templates/channels.html | 13 ++++---------
 1 file changed, 4 insertions(+), 9 deletions(-)

diff --git a/templates/channels.html b/templates/channels.html
index ce08c19..92c3b3c 100644
--- a/templates/channels.html
+++ b/templates/channels.html
@@ -26,10 +26,10 @@
                         </div>
                     </div>
                     <div class="channel-actions">
-                        <button class="btn btn-secondary" onclick="refreshChannel({{ channel.id }})">
+                        <button class="btn btn-secondary" onclick="refreshChannel({{ channel.id }}, event)">
                             Refresh Videos
                         </button>
-                        <button class="btn btn-danger" onclick="deleteChannel({{ channel.id }}, '{{ channel.title|replace("'", "\\'") }}')">
+                        <button class="btn btn-danger" onclick="deleteChannel({{ channel.id }}, '{{ channel.title|replace("'", "\\'") }}', event)">
                             Delete
                         </button>
                         <a href="/?channel={{ channel.id }}" class="btn btn-link">View Videos</a>
@@ -49,7 +49,7 @@
 
 {% block scripts %}
 <script>
-    function refreshChannel(channelId) {
+    function refreshChannel(channelId, event) {
         const button = event.target;
         button.disabled = true;
         button.textContent = 'Refreshing...';
@@ -75,11 +75,7 @@
         });
     }
 
-    function deleteChannel(channelId, channelTitle) {
-        if (!confirm(`Are you sure you want to delete "${channelTitle}"?\n\nThis will permanently remove:\n- The channel subscription\n- All video history\n- ALL downloaded video files for this channel\n\nThis action cannot be undone.`)) {
-            return;
-        }
-
+    function deleteChannel(channelId, channelTitle, event) {
         const button = event.target;
         const originalText = button.textContent;
         button.disabled = true;
@@ -91,7 +87,6 @@
         .then(response => response.json())
         .then(data => {
             if (data.status === 'success') {
-                alert(data.message);
                 location.reload();
             } else {
                 alert('Failed to delete: ' + data.message);