From 940b4075606a1cbb397324dfde339f98cfe83b8f Mon Sep 17 00:00:00 2001
From: Mike Mueller <mike.mueller@cstudio.ch>
Date: Fri, 1 May 2026 20:26:43 +0200
Subject: [PATCH] fix(admin): drop overflow-hidden from action-dropdown tables
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

The Users and Devices tables had `overflow-hidden` on the wrapper div for
clean rounded corners. That same clipping was hiding the bottom half of
the per-row action menu (a `<details>`/`<summary>` popover absolutely
positioned inside the last cell). Removing `overflow-hidden` lets the
dropdown extend past the table edge — the popover already has its own
border + shadow, so the loss in corner aesthetics is negligible.

The other read-only tables (audit, recordings, oidc, address_books) keep
`overflow-hidden` since they don't host popovers.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
---
 src/api/admin/pages/devices.rs | 5 ++++-
 src/api/admin/pages/users.rs   | 5 ++++-
 2 files changed, 8 insertions(+), 2 deletions(-)

diff --git a/src/api/admin/pages/devices.rs b/src/api/admin/pages/devices.rs
index 76539df..b713541 100644
--- a/src/api/admin/pages/devices.rs
+++ b/src/api/admin/pages/devices.rs
@@ -84,9 +84,12 @@ async fn render_table(state: &Arc<AppState>) -> Result<String, ApiError> {
         .await
         .map_err(|e| ApiError::Internal(e.to_string()))?;
     let mut s = String::new();
+    // No `overflow-hidden` on the table wrapper: the per-row action menu is
+    // an absolutely-positioned `<details>` popover inside a <td>, and the
+    // wrapper's clipping was hiding the bottom half of the menu.
     let _ = write!(
         s,
-        r##"<div class="rounded-md border border-slate-800 bg-slate-900 overflow-hidden">
+        r##"<div class="rounded-md border border-slate-800 bg-slate-900">
 <table class="w-full text-sm">
   <thead class="text-xs uppercase text-slate-500 bg-slate-950">
     <tr>
diff --git a/src/api/admin/pages/users.rs b/src/api/admin/pages/users.rs
index 9d9a276..6bc2c84 100644
--- a/src/api/admin/pages/users.rs
+++ b/src/api/admin/pages/users.rs
@@ -335,8 +335,11 @@ async fn render_table(state: &Arc<AppState>) -> Result<String, ApiError> {
         }
     }
     let mut s = String::new();
+    // No `overflow-hidden` on the table wrapper: the per-row action menu is
+    // an absolutely-positioned `<details>` popover inside a <td>, and the
+    // wrapper's clipping was hiding the bottom half of the menu.
     s.push_str(
-        r##"<div class="rounded-md border border-slate-800 bg-slate-900 overflow-hidden">
+        r##"<div class="rounded-md border border-slate-800 bg-slate-900">
 <table class="w-full text-sm">
   <thead class="text-xs uppercase text-slate-500 bg-slate-950">
     <tr>