From 517df5aa08ec368125636c42af3f1f203cc9f707 Mon Sep 17 00:00:00 2001
From: "google-labs-jules[bot]"
 <161369871+google-labs-jules[bot]@users.noreply.github.com>
Date: Mon, 23 Feb 2026 20:52:11 +0000
Subject: [PATCH] =?UTF-8?q?=F0=9F=8E=A8=20Palette:=20Enhance=20table=20sor?=
 =?UTF-8?q?ting=20accessibility=20and=20search=20UI?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

This change implements a micro-UX improvement for the inventory list views:
- Added keyboard navigation support (Enter/Space) and proper ARIA roles/attributes to table sorting headers.
- Added visual focus indicators to sortable headers for better keyboard accessibility.
- Updated the search bar to use modern DaisyUI 'join' layout for improved visual grouping.
- Fixed a syntax error in base.html (unbalanced HTML tags for QuickScan trigger).

♿ Accessibility:
- Added role="button", tabindex="0", and aria-label to sortable headers.
- Implemented aria-sort state updates for screen reader feedback.
- Fixed stray button tag to ensure a valid DOM tree.

🎯 Why:
Users relying on keyboard navigation were previously unable to sort tables. Screen reader users lacked feedback on the current sort state. The search UI was using deprecated DaisyUI components.

Co-authored-by: Woschj <81321922+Woschj@users.noreply.github.com>
---
 .Jules/palette.md                   |  4 ++++
 app/templates/base.html             |  2 +-
 app/templates/shared/list_base.html | 34 ++++++++++++++++++++++-------
 3 files changed, 31 insertions(+), 9 deletions(-)

diff --git a/.Jules/palette.md b/.Jules/palette.md
index 3ce56a2..1a248b0 100644
--- a/.Jules/palette.md
+++ b/.Jules/palette.md
@@ -7,3 +7,7 @@
 ## 2026-02-15 - [Semantic Form Labels]
 **Learning:** Even with clear visual labels, missing `for` attributes on `<label>` elements prevents proper association with inputs for assistive technologies.
 **Action:** Explicitly link labels and inputs using `for` and `id` attributes in all form components.
+
+## 2026-02-23 - [Micro-UX: Maintainable Diffs in Legacy Templates]
+**Learning:** When working with legacy templates that use CRLF line endings, performing full-file conversions (like `dos2unix`) results in massive, unreviewable diffs. For micro-UX improvements, it is critical to preserve original line endings to keep the PR focused on semantic changes.
+**Action:** Use `dos2unix` only temporarily for patching, or use targeted tools like `sed` to modify specific lines while respecting original formatting. Always verify diff size before submission.
diff --git a/app/templates/base.html b/app/templates/base.html
index d35052c..9d24adb 100755
--- a/app/templates/base.html
+++ b/app/templates/base.html
@@ -243,7 +243,7 @@
                     <!-- Scanner Animation -->
                     <div class="absolute w-full h-0.5 bg-primary/50 -top-1 left-0 animate-scanner after:content-[''] after:absolute after:top-0 after:left-0 after:w-full after:h-full after:bg-primary/30"></div>
                 </div>
-            </button>
+            </div>
             
             <!-- Trennlinie -->
             <div class="sidebar-divider"></div>
diff --git a/app/templates/shared/list_base.html b/app/templates/shared/list_base.html
index c89a7b4..f890eb6 100755
--- a/app/templates/shared/list_base.html
+++ b/app/templates/shared/list_base.html
@@ -8,14 +8,14 @@
             <!-- Suchleiste und Aktionen -->
             <div class="flex flex-col md:flex-row gap-4 items-center">
                 <div class="form-control flex-1">
-                    <label for="searchInput" class="sr-only">Suchen</label>
-                    <div class="input-group">
+                    <label for="searchInput" class="sr-only">Suchen</label>
+                    <div class="join">
                         <input type="text" 
                                placeholder="Suchen..." 
-                               class="input input-bordered w-full" 
-                               id="searchInput"
-                               aria-label="Suchen">
-                        <button type="button" class="btn btn-square" aria-label="Suche ausführen">
+                               class="input input-bordered w-full join-item"
+                               id="searchInput"
+                               aria-label="Suchen">
+                        <button type="button" class="btn btn-square join-item" aria-label="Suche ausführen">
                             <i class="fas fa-search"></i>
                         </button>
                     </div>
@@ -95,7 +95,12 @@ <h3 class="card-title text-lg">
     document.querySelectorAll('th').forEach(th => {
         if (!th.classList.contains('no-sort')) {
             const wrapper = document.createElement('div');
-            wrapper.className = 'flex items-center gap-2 cursor-pointer select-none';
+            wrapper.className = 'flex items-center gap-2 cursor-pointer select-none focus:outline-none focus:ring-2 focus:ring-primary/50 rounded px-1';
+            wrapper.setAttribute('role', 'button');
+            wrapper.setAttribute('tabindex', '0');
+            const label = th.textContent.trim();
+            wrapper.setAttribute('aria-label', `Sortieren nach ${label}`);
+
             wrapper.innerHTML = `
                 ${th.innerHTML}
                 <span class="sort-icons opacity-50">
@@ -105,9 +110,17 @@ <h3 class="card-title text-lg">
             th.innerHTML = '';
             th.appendChild(wrapper);
 
-            wrapper.addEventListener('click', () => {
+            const sortHandler = () => {
                 const column = Array.from(th.parentElement.children).indexOf(th);
                 sortTable(column, th);
+            };
+
+            wrapper.addEventListener('click', sortHandler);
+            wrapper.addEventListener('keydown', (e) => {
+                if (e.key === 'Enter' || e.key === ' ') {
+                    e.preventDefault();
+                    sortHandler();
+                }
             });
         }
     });
@@ -117,6 +130,9 @@ <h3 class="card-title text-lg">
         const tbody = table.querySelector('tbody');
         const rows = Array.from(tbody.querySelectorAll('tr'));
         
+        // Reset all aria-sort
+        table.querySelectorAll('th').forEach(header => header.removeAttribute('aria-sort'));
+
         // Update Sort-Direction
         if (currentSort.column === column) {
             currentSort.direction = currentSort.direction === 'asc' ? 'desc' : 'asc';
@@ -125,6 +141,8 @@ <h3 class="card-title text-lg">
             currentSort.direction = 'asc';
         }
 
+        th.setAttribute('aria-sort', currentSort.direction === 'asc' ? 'ascending' : 'descending');
+
         // Update Sort-Icons
         table.querySelectorAll('.sort-icons').forEach(icon => {
             icon.innerHTML = '<i class="fas fa-sort"></i>';