From 88b73f663f543d6f1403f4b79b886a78a848bbc6 Mon Sep 17 00:00:00 2001
From: bruno <bruno.charest@gmail.com>
Date: Fri, 24 Jul 2026 10:04:13 -0400
Subject: [PATCH] fix: header logout button dynamically recreated by
 renderUserMenu

- renderUserMenu() rebuilds #logout-btn with innerHTML, wiping inline onclick
- Added onclick='window.handleLogout()' to the dynamic innerHTML string
- Restored addEventListener as double safety
- Settings #profile-logout already has onclick to handleLogout
---
 frontend/js/auth.js | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/frontend/js/auth.js b/frontend/js/auth.js
index 4c983ab..6c2d8f4 100644
--- a/frontend/js/auth.js
+++ b/frontend/js/auth.js
@@ -203,11 +203,11 @@ const AuthManager = {
       userMenu.innerHTML = "";
       return;
     }
-    userMenu.innerHTML = '<span class="user-display-name">' + (user.display_name || user.username) + "</span>" + '<button class="btn-logout" id="logout-btn" title="' + t('auth.logout_title') + '"><i data-lucide="log-out" style="width:14px;height:14px"></i></button>';
+    userMenu.innerHTML = '<span class="user-display-name">' + (user.display_name || user.username) + "</span>" + '<button class="btn-logout" id="logout-btn" title="' + t('auth.logout_title') + '" onclick="window.handleLogout()"><i data-lucide="log-out" style="width:14px;height:14px"></i></button>';
     safeCreateIcons();
 
     const logoutBtn = document.getElementById("logout-btn");
-    // Logout button — handled inline in index.html
+    if (logoutBtn) logoutBtn.addEventListener("click", () => AuthManager.logout());
 
     const adminRow = document.getElementById("admin-menu-row");
     if (adminRow) {