From ac56974e8339649242a9d29d23c617220e8b2a10 Mon Sep 17 00:00:00 2001
From: Lars <Lars@stommer.de>
Date: Fri, 20 Mar 2026 08:26:18 +0100
Subject: [PATCH] fix: make buttons always visible in AdminUserRestrictionsPage
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Bottom bar changes:
- Always visible when user selected (not hidden)
- Buttons disabled when no changes (clearer state)
- Moved outside inner block to prevent hiding

Action column changes:
- "↺ Zurück" button always visible per feature
- Disabled when no override exists (grayed out)
- Consistent button presence improves UX

This fixes the issue where buttons were not shown
because they were conditionally rendered.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
---
 .../src/pages/AdminUserRestrictionsPage.jsx   | 73 ++++++++++---------
 1 file changed, 40 insertions(+), 33 deletions(-)

diff --git a/frontend/src/pages/AdminUserRestrictionsPage.jsx b/frontend/src/pages/AdminUserRestrictionsPage.jsx
index 55cedb6..9364526 100644
--- a/frontend/src/pages/AdminUserRestrictionsPage.jsx
+++ b/frontend/src/pages/AdminUserRestrictionsPage.jsx
@@ -434,15 +434,19 @@ export default function AdminUserRestrictionsPage() {
 
                           {/* Action */}
                           <td style={{ padding: '12px 16px', textAlign: 'right' }}>
-                            {override && (
-                              <button
-                                className="btn btn-secondary"
-                                onClick={() => handleChange(feature.id, '')}
-                                style={{ padding: '4px 8px', fontSize: 11 }}
-                              >
-                                ↺ Zurück zu Standard
-                              </button>
-                            )}
+                            <button
+                              className="btn btn-secondary"
+                              onClick={() => handleChange(feature.id, '')}
+                              disabled={!override}
+                              style={{
+                                padding: '4px 8px',
+                                fontSize: 11,
+                                opacity: override ? 1 : 0.4,
+                                cursor: override ? 'pointer' : 'not-allowed'
+                              }}
+                            >
+                              ↺ Zurück
+                            </button>
                           </td>
                         </tr>
                       )
@@ -453,30 +457,6 @@ export default function AdminUserRestrictionsPage() {
             </table>
           </div>
 
-          {/* Fixed Bottom Bar */}
-          <div style={{
-            position: 'fixed', bottom: 0, left: 0, right: 0,
-            background: 'var(--bg)', borderTop: '1px solid var(--border)',
-            padding: 16, display: 'flex', gap: 8, zIndex: 100,
-            boxShadow: '0 -2px 10px rgba(0,0,0,0.1)'
-          }}>
-            <button
-              className="btn btn-secondary"
-              onClick={() => setChanges({})}
-              disabled={!hasChanges || saving}
-              style={{ flex: 1 }}
-            >
-              <X size={14} /> Abbrechen
-            </button>
-            <button
-              className="btn btn-primary"
-              onClick={handleSave}
-              disabled={!hasChanges || saving}
-              style={{ flex: 2 }}
-            >
-              {saving ? 'Speichern...' : hasChanges ? `${Object.keys(changes).length} Änderung(en) speichern` : 'Keine Änderungen'}
-            </button>
-          </div>
 
           {/* Legend */}
           <div style={{
@@ -493,6 +473,33 @@ export default function AdminUserRestrictionsPage() {
           </div>
         </>
       )}
+
+      {/* Fixed Bottom Bar - Always visible when user selected */}
+      {selectedUser && (
+        <div style={{
+          position: 'fixed', bottom: 0, left: 0, right: 0,
+          background: 'var(--bg)', borderTop: '1px solid var(--border)',
+          padding: 16, display: 'flex', gap: 8, zIndex: 100,
+          boxShadow: '0 -2px 10px rgba(0,0,0,0.1)'
+        }}>
+          <button
+            className="btn btn-secondary"
+            onClick={() => setChanges({})}
+            disabled={!hasChanges || saving}
+            style={{ flex: 1 }}
+          >
+            <X size={14} /> Abbrechen
+          </button>
+          <button
+            className="btn btn-primary"
+            onClick={handleSave}
+            disabled={!hasChanges || saving}
+            style={{ flex: 2 }}
+          >
+            {saving ? 'Speichern...' : hasChanges ? `${Object.keys(changes).length} Änderung(en) speichern` : 'Keine Änderungen'}
+          </button>
+        </div>
+      )}
     </div>
   )
 }