From 12d3e697e459c82883d784c6510b8f30b6cc988e Mon Sep 17 00:00:00 2001
From: Brad Bodine <bbodine1@gmail.com>
Date: Mon, 5 Aug 2024 16:02:19 -0500
Subject: [PATCH] Refactor the faa eligibility results accordion UI navigation.
 (#4211)

---
 ...ibility_results_per_tax_household.html.erb | 46 +++++++++++--------
 1 file changed, 26 insertions(+), 20 deletions(-)

diff --git a/components/financial_assistance/app/views/financial_assistance/applications/_eligibility_results_per_tax_household.html.erb b/components/financial_assistance/app/views/financial_assistance/applications/_eligibility_results_per_tax_household.html.erb
index e6c0168d89f..8eecd098dd8 100644
--- a/components/financial_assistance/app/views/financial_assistance/applications/_eligibility_results_per_tax_household.html.erb
+++ b/components/financial_assistance/app/views/financial_assistance/applications/_eligibility_results_per_tax_household.html.erb
@@ -161,27 +161,33 @@
 
     <p data-cuke='your_application_reference_2' class="mt-4"><%= l10n("faa.results.your_application_reference_2", application_hbx_id: @application.hbx_id)%></p>
 
-    <div class="mt-4 accordion card" data-cuke='other_actions'>
-
-      <h3 class="accordion-header collapsed" data-toggle="collapse" data-target="#fa_other_actions" aria-expanded="true" aria-controls="fa_other_actions" href="#fa_other_actions">
-        <%= l10n("other_actions") %>
-      </h3>
-
-      <div id="fa_other_actions" class="collapse">
-        <div class="panel-group" id="accordion">
-          <% if FinancialAssistanceRegistry.feature_enabled?(:fa_send_to_external_verification) && !@application.is_transferrable? %>
-            <div class="w-100 pb-4">
-              <p><%= l10n("faa.full_long_name_determination",
-                  program_long_name: FinancialAssistanceRegistry[:medicaid_or_chip_agency_long_name].setting(:name).item,
-                  program_short_name: FinancialAssistanceRegistry[:medicaid_or_chip_program_short_name].setting(:name).item) %></p>
-              <%= h(link_to l10n("faa.send_to_external_verification"), update_transfer_requested_application_url, method: :put, class: "button secondary", id: "btn-request-transfer", remote: true) %>
+    <div id="other-accordion" data-cuke='other_actions'>
+      <div class="card">
+        <div class="card-header" id="otherActionsHeading">
+          <h5 class="mb-0">
+            <button class="btn btn-link" data-toggle="collapse" data-target="#otherActions" aria-expanded="true" aria-controls="otherActions">
+              <%= l10n("other_actions") %>
+            </button>
+          </h5>
+        </div>
+
+        <div id="otherActions" class="collapse show" aria-labelledby="otherActionsHeading" data-parent="#other-accordion">
+          <div class="card-body">
+            <% if FinancialAssistanceRegistry.feature_enabled?(:fa_send_to_external_verification) && !@application.is_transferrable? %>
+              <div class="w-100 pb-4">
+                <p><%= l10n("faa.full_long_name_determination",
+                    program_long_name: FinancialAssistanceRegistry[:medicaid_or_chip_agency_long_name].setting(:name).item,
+                    program_short_name: FinancialAssistanceRegistry[:medicaid_or_chip_program_short_name].setting(:name).item) %></p>
+                <%= h(link_to l10n("faa.send_to_external_verification"), update_transfer_requested_application_url, method: :put, class: "button secondary", id: "btn-request-transfer", remote: true) %>
+              </div>
+            <% end %>
+
+            <div class="w-100 pb-2">
+              <p><%= l10n("faa.do_not_agree",
+                appeal_link: EnrollRegistry[:enroll_app].setting(:appeal_link).item,
+                find_expert_link: EnrollRegistry[:enroll_app].setting(:find_expert_link).item) %></p>
+              <%= h(link_to l10n("faa.results.view_my_applications"), applications_path, class: "button secondary") %>
             </div>
-          <% end %>
-          <div class="w-100 pb-2">
-            <p><%= l10n("faa.do_not_agree",
-              appeal_link: EnrollRegistry[:enroll_app].setting(:appeal_link).item,
-              find_expert_link: EnrollRegistry[:enroll_app].setting(:find_expert_link).item) %></p>
-            <%= h(link_to l10n("faa.results.view_my_applications"), applications_path, class: "button secondary") %>
           </div>
         </div>
       </div>