From 88ee55d603bcdb9ba8343e10a3c6044ecd7c00fe Mon Sep 17 00:00:00 2001
From: Graham McIntire <graham@mcintire.me>
Date: Fri, 13 Mar 2026 10:45:45 -0500
Subject: [PATCH] fix: show device link dropdown inline below clicked row in
 Preseem devices table

The linking row was rendered in a separate loop after all device rows,
causing it to always appear at the bottom of the table. Changed to use
a single for loop with an inline if check so the linking row appears
immediately after the corresponding device row.
---
 lib/towerops_web/live/org/preseem_devices_live.html.heex | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/lib/towerops_web/live/org/preseem_devices_live.html.heex b/lib/towerops_web/live/org/preseem_devices_live.html.heex
index 3f5938f0..edcbb62a 100644
--- a/lib/towerops_web/live/org/preseem_devices_live.html.heex
+++ b/lib/towerops_web/live/org/preseem_devices_live.html.heex
@@ -155,8 +155,8 @@
             </tr>
           </thead>
           <tbody class="divide-y divide-gray-200 bg-white dark:divide-white/5 dark:bg-transparent">
+            <%= for ap <- @access_points do %>
             <tr
-              :for={ap <- @access_points}
               id={"ap-row-#{ap.id}"}
               class="hover:bg-gray-50 dark:hover:bg-white/5"
             >
@@ -224,7 +224,7 @@
               </td>
             </tr>
             <%!-- Inline linking row --%>
-            <%= for ap <- @access_points, ap.id == @linking_ap_id do %>
+            <%= if ap.id == @linking_ap_id do %>
               <tr id={"linking-row-#{ap.id}"} class="bg-indigo-50 dark:bg-indigo-900/20">
                 <td colspan="6" class="px-4 py-4">
                   <div class="flex items-center gap-4">
@@ -292,6 +292,7 @@
                 </td>
               </tr>
             <% end %>
+            <% end %>
           </tbody>
         </table>
       </div>