cleanup

src/registrar/assets/js/get-gov.js

@@ -17,6 +17,22 @@ var SUCCESS = "success";
 // <<>><<>><<>><<>><<>><<>><<>><<>><<>><<>><<>><<>><<>><<>><<>>
 // Helper functions.
 
+/**
+ * Hide element
+ *
+*/
+const hideElement = (element) => {
+  element.classList.add('display-none');
+};
+
+/**
+ * Show element
+ *
+*/
+const showElement = (element) => {
+  element.classList.remove('display-none');
+};
+
 /** Makes an element invisible. */
 function makeHidden(el) {
   el.style.position = "absolute";
@@ -1019,22 +1035,6 @@ function updatePagination(itemName, paginationSelector, counterSelector, headerA
   }
 }
 
-/**
- * Hide element
- *
-*/
-const hideElement = (element) => {
-  element.classList.add('display-none');
-};
-
-/**
- * Show element
- *
-*/
-const showElement = (element) => {
-  element.classList.remove('display-none');
-};
-
 /**
  * A helper that toggles content/ no content/ no search results
  *
@@ -1071,7 +1071,7 @@ const updateDisplay = (data, dataWrapper, noDataWrapper, noSearchResultsWrapper,
 const unsetHeader = (header) => {
   header.removeAttribute('aria-sort');
   let headerName = header.innerText;
-  const headerLabel = `${headerName}', sortable column, currently unsorted"`;
+  const headerLabel = `${headerName}, sortable column, currently unsorted"`;
   const headerButtonLabel = `Click to sort by ascending order.`;
   header.setAttribute("aria-label", headerLabel);
   header.querySelector('.usa-table__header__button').setAttribute("title", headerButtonLabel);
@@ -1219,11 +1219,11 @@ document.addEventListener('DOMContentLoaded', function() {
         hideElement(resetButton);
       }
       loadDomains(1, 'id', 'asc');
-      resetheaders();
+      resetHeaders();
     })
 
     // Reset UI and accessibility
-    function resetheaders() {
+    function resetHeaders() {
       tableHeaders.forEach(header => {
         // Unset sort UI in headers
         unsetHeader(header);
@@ -1237,7 +1237,7 @@ document.addEventListener('DOMContentLoaded', function() {
       currentSearchTerm = '';
       hideElement(resetButton);
       loadDomains(1, 'id', 'asc', hasLoaded, '');
-      resetheaders();
+      resetHeaders();
     }
 
     if (resetButton) {
@@ -1512,9 +1512,9 @@ document.addEventListener('DOMContentLoaded', function() {
             const closeButton = modal.querySelector('.usa-modal__close');
             submitButton.addEventListener('click', function() {
               pk = submitButton.getAttribute('data-pk');
-              // Close the modal to remove the USWDS UI locl classes
+              // Close the modal to remove the USWDS UI local classes
               closeButton.click();
-              // If we're deleteing the last item on a page that is not page 1, we'll need to refresh the display to the previous page
+              // If we're deleting the last item on a page that is not page 1, we'll need to refresh the display to the previous page
               let pageToDisplay = data.page;
               if (data.total == 1 && data.unfiltered_total > 1) {
                 pageToDisplay--;
@@ -1573,11 +1573,11 @@ document.addEventListener('DOMContentLoaded', function() {
         hideElement(resetButton);
       }
       loadDomainRequests(1, 'id', 'asc');
-      resetheaders();
+      resetHeaders();
     })
 
     // Reset UI and accessibility
-    function resetheaders() {
+    function resetHeaders() {
       tableHeaders.forEach(header => {
         // unset sort UI in headers
         unsetHeader(header);
@@ -1591,7 +1591,7 @@ document.addEventListener('DOMContentLoaded', function() {
       currentSearchTerm = '';
       hideElement(resetButton);
       loadDomainRequests(1, 'id', 'asc', hasLoaded, '');
-      resetheaders();
+      resetHeaders();
     }
 
     if (resetButton) {

src/registrar/templates/home.html

@@ -23,9 +23,6 @@
     </a>
   </p>
 
-  <!-- CSRF token -->
-  {% csrf_token %}
-
   <section class="section--outlined domains">
     <div class="grid-row">
       <div class="mobile:grid-col-12 desktop:grid-col-6">

src/registrar/tests/test_views_domains_json.py

@@ -105,7 +105,7 @@ class GetDomainsJsonTest(TestWithUser, WebTest):
     def test_get_domains_json_search(self):
         """Test search."""
         # Define your URL variables as a dictionary
-        url_vars = {"search_term": "e2"}  # Modify with your actual variables
+        url_vars = {"search_term": "e2"}
 
         # Use the params parameter to include URL variables
         response = self.app.get(reverse("get_domains_json"), params=url_vars)

src/registrar/tests/test_views_requests_json.py

@@ -205,7 +205,7 @@ class GetRequestsJsonTest(TestWithUser, WebTest):
     def test_get_domain_requests_json_search(self):
         """Test search."""
         # Define your URL variables as a dictionary
-        url_vars = {"search_term": "lamb"}  # Modify with your actual variables
+        url_vars = {"search_term": "lamb"}
 
         # Use the params parameter to include URL variables
         response = self.app.get(reverse("get_domain_requests_json"), params=url_vars)
@@ -234,7 +234,7 @@ class GetRequestsJsonTest(TestWithUser, WebTest):
     def test_get_domain_requests_json_search_new_domains(self):
         """Test search when looking up New domain requests"""
         # Define your URL variables as a dictionary
-        url_vars = {"search_term": "ew"}  # Modify with your actual variables
+        url_vars = {"search_term": "ew"}
 
         # Use the params parameter to include URL variables
         response = self.app.get(reverse("get_domain_requests_json"), params=url_vars)