From a05be515bc9f674d8b905e915a5ba0da33b52eeb Mon Sep 17 00:00:00 2001
From: maebeale <maebeale@users.noreply.github.com>
Date: Mon, 22 Jun 2026 10:52:11 -0400
Subject: [PATCH] Make each registration form submission card a full link

Wrap the entire card (icon + org/title) in the link to the submission's
public view so the whole row is clickable, not just the small green icon.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
---
 .../link_organization.html.erb                | 19 ++++++++++---------
 1 file changed, 10 insertions(+), 9 deletions(-)

diff --git a/app/views/event_registrations/link_organization.html.erb b/app/views/event_registrations/link_organization.html.erb
index 53520ca02c..6bb65e84fe 100644
--- a/app/views/event_registrations/link_organization.html.erb
+++ b/app/views/event_registrations/link_organization.html.erb
@@ -43,20 +43,21 @@
           <% multiple = present_entries.size > 1 %>
           <ul class="space-y-3">
             <% present_entries.each_with_index do |entry, i| %>
-              <li class="flex items-start gap-2">
-                <%# Same green form-submission icon used on the registrants list, linking
-                    to this specific submission's public view. %>
+              <li>
+                <%# The whole card links to this specific submission's public view;
+                    the green form-submission icon (also used on the registrants list)
+                    sits inline as the affordance. %>
                 <%= link_to event_public_registration_path(@event_registration.event, **form_show_params, form_submission_id: entry[:submission].id, return_to: "link_organization", link_org_return_to: params[:return_to]),
                             target: "_blank", rel: "noopener",
-                            class: "text-green-600 hover:text-green-800 mt-1 shrink-0",
+                            class: "flex items-start gap-2 group hover:text-green-800",
                             title: multiple ? "View submission ##{i + 1}" : "View form submission",
                             data: { turbo_frame: "_top" } do %>
-                  <i class="fa-solid fa-file-lines"></i>
+                  <i class="fa-solid fa-file-lines text-green-600 group-hover:text-green-800 mt-1 shrink-0"></i>
+                  <div class="min-w-0">
+                    <p class="text-base text-gray-800">Organization: <strong><%= entry[:org_name].presence || "—" %></strong><% if entry[:organization]&.city_state.present? %> <span class="text-sm text-gray-500">· <%= entry[:organization].city_state %></span><% end %></p>
+                    <p class="text-base text-gray-800">Position / title: <strong><%= entry[:position].presence || "—" %></strong></p>
+                  </div>
                 <% end %>
-                <div class="min-w-0">
-                  <p class="text-base text-gray-800">Organization: <strong><%= entry[:org_name].presence || "—" %></strong><% if entry[:organization]&.city_state.present? %> <span class="text-sm text-gray-500">· <%= entry[:organization].city_state %></span><% end %></p>
-                  <p class="text-base text-gray-800">Position / title: <strong><%= entry[:position].presence || "—" %></strong></p>
-                </div>
               </li>
             <% end %>
           </ul>