Refactor personal form JavaScript: move inline script to separate file (

#4881)
ideacrew · Jan 10, 2025 · 4a84714 · 4a84714
1 parent b4e057b
commit 4a84714
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 19 deletions.
diff --git a/app/assets/javascripts/inline_replacements.js b/app/assets/javascripts/inline_replacements.js
@@ -231,3 +231,21 @@ function init_dependent_form() {
     }
   }
 }
+
+$('.remove-personal-form').off('click');
+$('.remove-personal-form').on('click', function (e) {
+  $(".append_consumer_info").empty();
+  if ($(".my-household-page")) {
+    $("#person-<%= @person.id %>").addClass('hidden');
+    $("a[id^=edit-member]").removeClass('disabled');
+    $("a[id^=edit-member]").removeAttr('disabled');
+    // primary_family
+    var family_table = $("<%= escape_javascript(render 'insured/families/manage_family_table', members: @person.primary_family.active_family_members) %>");
+    $('#manage_family_content').html(family_table);
+  }
+});
+
+$(document).off('click', '.confirm-member');
+$(document).on('click', '.confirm-member', function(e) {
+  $('#save_personal').trigger('click');
+});
diff --git a/app/views/people/landing_pages/_personal.html.erb b/app/views/people/landing_pages/_personal.html.erb
@@ -128,25 +128,6 @@
       </span>
     </div>
   <% end %>
-  <script>
-    $('.remove-personal-form').off('click');
-    $('.remove-personal-form').on('click', function (e) {
-      $(".append_consumer_info").empty();
-      if ($(".my-household-page")) {
-        $("#person-<%= @person.id %>").addClass('hidden');
-        $("a[id^=edit-member]").removeClass('disabled');
-        $("a[id^=edit-member]").removeAttr('disabled');
-        // primary_family
-        var family_table = $("<%= escape_javascript(render 'insured/families/manage_family_table', members: @person.primary_family.active_family_members) %>");
-        $('#manage_family_content').html(family_table);
-      }
-    });
-
-    $(document).off('click', '.confirm-member');
-    $(document).on('click', '.confirm-member', function(e) {
-      $("#save_personal").click();
-    });
-  </script>
 
 <% else %>
   <%= form_for @person do |f|%>