From 7499c380378bd3079ff9dceee71a67284943d5b0 Mon Sep 17 00:00:00 2001
From: Bryan Van Deusen <bryan.vandeusen@fabledsword.com>
Date: Wed, 7 Oct 2026 23:36:26 -0400
Subject: [PATCH] feat(ansible): the whole run row opens the run (#5214)

Same .row-link / .row-target as the Alerts rules list (snippet #5320).

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
---
 steward/templates/ansible/_runs_page.html | 7 ++++---
 tests/core/test_ansible_runs_page.py      | 3 ++-
 2 files changed, 6 insertions(+), 4 deletions(-)

diff --git a/steward/templates/ansible/_runs_page.html b/steward/templates/ansible/_runs_page.html
index c97ce1e..2c479ec 100644
--- a/steward/templates/ansible/_runs_page.html
+++ b/steward/templates/ansible/_runs_page.html
@@ -1,11 +1,12 @@
 {# ansible/_runs_page.html — one page of runs, then the sentinel that loads the
-   next page as the reader nears it (core.paging, pref 172). #}
+   next page as the reader nears it (core.paging, pref 172). The whole row
+   opens the run (.row-link); its title is the one real link. #}
 {% from "_macros.html" import status_badge, scroll_more, scroll_end %}
     {% for row in runs %}
     {% set run = row.run %}
-    <tr>
+    <tr class="row-link">
         <td>
-            <a href="/ansible/runs/{{ run.id }}">{{ row.title }}</a>
+            <a class="row-target" href="/ansible/runs/{{ run.id }}">{{ row.title }}</a>
             {% if row.check %}<span class="badge badge-dim" title="Ansible check mode: nothing was changed">dry run</span>{% endif %}
             <div class="field-hint">{{ run.source_name }}</div>
         </td>
diff --git a/tests/core/test_ansible_runs_page.py b/tests/core/test_ansible_runs_page.py
index 03d3be1..66f3620 100644
--- a/tests/core/test_ansible_runs_page.py
+++ b/tests/core/test_ansible_runs_page.py
@@ -54,7 +54,8 @@ async def test_the_runs_list_names_each_run_with_one_badge(app):
         page = await render_template("ansible/index.html", runs=runs, after="2026-10-07T01:00:00+00:00|x")
 
     main = page[page.index("<main"):]
-    assert '<a href="/ansible/runs/aaaa1111">update.yml on web-1</a>' in main
+    assert '<tr class="row-link">' in main                                  # the whole row opens the run
+    assert '<a class="row-target" href="/ansible/runs/aaaa1111">update.yml on web-1</a>' in main
     assert "steward:target:t1" not in main                     # no raw scope, not even as a tooltip
     assert ">update.yml on all inventory targets</a>" in main
     assert '<span class="badge badge-info">Running</span>' in main