From 77e5a544d18a716cf11eb44b8902fe8d6e577774 Mon Sep 17 00:00:00 2001 From: Erik Date: Thu, 9 Apr 2026 11:28:28 +0200 Subject: [PATCH] fix: restore sort click to full th area, not just label span MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Sort listener was on .th-label span only, making clickable area too small. Moved back to full .sortable th element. The × hide button already uses stopPropagation so it won't trigger sort. Co-Authored-By: Claude Opus 4.6 (1M context) --- static/inventory.js | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/static/inventory.js b/static/inventory.js index 501ec610..225d52e3 100644 --- a/static/inventory.js +++ b/static/inventory.js @@ -589,11 +589,11 @@ function displayResults(data) { searchResults.innerHTML = html; - // Add click event listeners to sortable headers (click label to sort) - document.querySelectorAll('.sortable .th-label').forEach(label => { - label.style.cursor = 'pointer'; - label.addEventListener('click', () => { - const sortField = label.closest('th').getAttribute('data-sort'); + // Add click event listeners to sortable headers (full th area sorts) + document.querySelectorAll('.sortable').forEach(header => { + header.style.cursor = 'pointer'; + header.addEventListener('click', () => { + const sortField = header.getAttribute('data-sort'); handleSort(sortField); }); });