From 7938bbe8e2f3983e04d2e6708904f018dfecb17e Mon Sep 17 00:00:00 2001
From: CocoByte <nicolle.leclair@gmail.com>
Date: Fri, 3 Jan 2025 12:26:46 -0700
Subject: [PATCH 1/5] Combine action column and delete columns.  Make "Action"
 header visible

---
 .../src/js/getgov/table-domain-requests.js    | 20 ++++++++++---------
 .../includes/domain_requests_table.html       |  2 +-
 .../templates/includes/domains_table.html     |  2 +-
 3 files changed, 13 insertions(+), 11 deletions(-)

diff --git a/src/registrar/assets/src/js/getgov/table-domain-requests.js b/src/registrar/assets/src/js/getgov/table-domain-requests.js
index 51e4ea12b..3078a0fbf 100644
--- a/src/registrar/assets/src/js/getgov/table-domain-requests.js
+++ b/src/registrar/assets/src/js/getgov/table-domain-requests.js
@@ -98,7 +98,7 @@ export class DomainRequestsTable extends BaseTable {
     }
 
     if (request.is_deletable) {
-      // 1st path: Just a modal trigger in any screen size for non-org users
+      // 1st path (non-org): Just a modal trigger in any screen size for non-org users
       modalTrigger = `
         <a 
           role="button" 
@@ -116,7 +116,7 @@ export class DomainRequestsTable extends BaseTable {
       // Request is deletable, modal and modalTrigger are built. Now check if we are on the portfolio requests page (by seeing if there is a portfolio value) and enhance the modalTrigger accordingly
       if (this.portfolioValue) {
 
-        // 2nd path: Just a modal trigger on mobile for org users or kebab + accordion with nested modal trigger on desktop for org users
+        // 2nd path (org model): Just a modal trigger on mobile for org users or kebab + accordion with nested modal trigger on desktop for org users
         modalTrigger = generateKebabHTML('delete-domain', request.id, 'Delete', domainName);
       }
     }
@@ -134,14 +134,16 @@ export class DomainRequestsTable extends BaseTable {
         ${request.status}
       </td>
       <td>
-        <a href="${actionUrl}">
-          <svg class="usa-icon" aria-hidden="true" focusable="false" role="img" width="24">
-            <use xlink:href="/public/img/sprite.svg#${request.svg_icon}"></use>
-          </svg>
-          ${actionLabel} <span class="usa-sr-only">${request.requested_domain ? request.requested_domain : 'New domain request'}</span>
-        </a>
+        <div class="grid-row grid-gap">
+          <a href="${actionUrl}">
+            <svg class="usa-icon" aria-hidden="true" focusable="false" role="img" width="24">
+              <use xlink:href="/public/img/sprite.svg#${request.svg_icon}"></use>
+            </svg>
+            ${actionLabel} <span class="usa-sr-only">${request.requested_domain ? request.requested_domain : 'New domain request'}</span>
+          </a>
+          ${customTableOptions.needsAdditionalColumn ? modalTrigger : ''}
+        </div>
       </td>
-      ${customTableOptions.needsAdditionalColumn ? '<td>'+modalTrigger+'</td>' : ''}
     `;
     tbody.appendChild(row);
     if (request.is_deletable) DomainRequestsTable.addDomainRequestsModal(request.requested_domain, request.id, request.created_at, tbody);
diff --git a/src/registrar/templates/includes/domain_requests_table.html b/src/registrar/templates/includes/domain_requests_table.html
index 56cdc2cec..9ab65ef7b 100644
--- a/src/registrar/templates/includes/domain_requests_table.html
+++ b/src/registrar/templates/includes/domain_requests_table.html
@@ -186,7 +186,7 @@
                     <th data-sortable="creator" scope="col" role="columnheader">Created by</th>
                     {% endif %}
                     <th data-sortable="status" scope="col" role="columnheader">Status</th>
-                    <th scope="col" role="columnheader"><span class="usa-sr-only">Action</span></th>
+                    <th scope="col" role="columnheader">Action</th>
                     <!-- AJAX will conditionally add a th for delete actions -->
                 </tr>
             </thead>
diff --git a/src/registrar/templates/includes/domains_table.html b/src/registrar/templates/includes/domains_table.html
index 15becea7a..db9f10087 100644
--- a/src/registrar/templates/includes/domains_table.html
+++ b/src/registrar/templates/includes/domains_table.html
@@ -214,7 +214,7 @@
             scope="col" 
             role="columnheader" 
           >
-            <span class="usa-sr-only">Action</span>
+            Action
           </th>
         </tr>
       </thead>

From 581dc7081d5a5946b51807c82841ee3df0b24938 Mon Sep 17 00:00:00 2001
From: CocoByte <nicolle.leclair@gmail.com>
Date: Mon, 6 Jan 2025 13:37:11 -0700
Subject: [PATCH 2/5] Make non-org Domain and Domain Request table columns
 align

---
 .../src/js/getgov/table-domain-requests.js    | 24 ++++---------------
 .../assets/src/js/getgov/table-domains.js     |  6 ++++-
 .../assets/src/sass/_theme/_base.scss         |  4 ++++
 3 files changed, 14 insertions(+), 20 deletions(-)

diff --git a/src/registrar/assets/src/js/getgov/table-domain-requests.js b/src/registrar/assets/src/js/getgov/table-domain-requests.js
index 3078a0fbf..3d5d114c3 100644
--- a/src/registrar/assets/src/js/getgov/table-domain-requests.js
+++ b/src/registrar/assets/src/js/getgov/table-domain-requests.js
@@ -53,24 +53,6 @@ export class DomainRequestsTable extends BaseTable {
     this.toggleExportButton(data.domain_requests);
 
     let needsDeleteColumn = data.domain_requests.some(request => request.is_deletable);
-
-    // Remove existing delete th and td if they exist
-    let existingDeleteTh =  document.querySelector('.delete-header');
-    if (!needsDeleteColumn) {
-      if (existingDeleteTh)
-        existingDeleteTh.remove();
-    } else {
-      if (!existingDeleteTh) {
-        const delheader = document.createElement('th');
-        delheader.setAttribute('scope', 'col');
-        delheader.setAttribute('role', 'columnheader');
-        delheader.setAttribute('class', 'delete-header width-5');
-        delheader.innerHTML = `
-          <span class="usa-sr-only">Delete Action</span>`;
-        let tableHeaderRow = this.tableWrapper.querySelector('thead tr');
-        tableHeaderRow.appendChild(delheader);
-      }
-    }
     return { 'needsAdditionalColumn': needsDeleteColumn };
   }
 
@@ -88,8 +70,12 @@ export class DomainRequestsTable extends BaseTable {
     <span class="usa-sr-only">Domain request cannot be deleted now. Edit the request for more information.</span>`;
 
     let markupCreatorRow = '';
+    
+    // Forces columns of the domain request and domain tables to align in non-org views
+    let columnWidthLimiterClass = 'width-quarter';
 
     if (this.portfolioValue) {
+      columnWidthLimiterClass = '';
       markupCreatorRow = `
         <td>
             <span class="text-wrap break-word">${request.creator ? request.creator : ''}</span>
@@ -133,7 +119,7 @@ export class DomainRequestsTable extends BaseTable {
       <td data-label="Status">
         ${request.status}
       </td>
-      <td>
+      <td class="${columnWidthLimiterClass}">
         <div class="grid-row grid-gap">
           <a href="${actionUrl}">
             <svg class="usa-icon" aria-hidden="true" focusable="false" role="img" width="24">
diff --git a/src/registrar/assets/src/js/getgov/table-domains.js b/src/registrar/assets/src/js/getgov/table-domains.js
index a6373a5c2..deeb04988 100644
--- a/src/registrar/assets/src/js/getgov/table-domains.js
+++ b/src/registrar/assets/src/js/getgov/table-domains.js
@@ -23,8 +23,12 @@ export class DomainsTable extends BaseTable {
     const row = document.createElement('tr');
 
     let markupForSuborganizationRow = '';
+    
+    // Forces columns of the domain request and domain tables to align in non-org views
+    let columnWidthLimiterClass = 'width-quarter';
 
     if (this.portfolioValue) {
+      columnWidthLimiterClass = ''
       markupForSuborganizationRow = `
         <td>
             <span class="text-wrap" aria-label="${domain.suborganization ? suborganization : 'No suborganization'}">${suborganization}</span>
@@ -55,7 +59,7 @@ export class DomainsTable extends BaseTable {
         </svg>
       </td>
       ${markupForSuborganizationRow}
-      <td>
+      <td class="${columnWidthLimiterClass}">
         <a href="${actionUrl}">
           <svg class="usa-icon" aria-hidden="true" focusable="false" role="img" width="24">
             <use xlink:href="/public/img/sprite.svg#${domain.svg_icon}"></use>
diff --git a/src/registrar/assets/src/sass/_theme/_base.scss b/src/registrar/assets/src/sass/_theme/_base.scss
index d73becd75..97f5710e9 100644
--- a/src/registrar/assets/src/sass/_theme/_base.scss
+++ b/src/registrar/assets/src/sass/_theme/_base.scss
@@ -281,4 +281,8 @@ abbr[title] {
 
 .maxw-fit-content {
   max-width: fit-content;
+}
+
+.width-quarter {
+  width: 25%;
 }
\ No newline at end of file

From 018967e0ac8186770ace3a1a2c99956165d69cc1 Mon Sep 17 00:00:00 2001
From: CocoByte <nicolle.leclair@gmail.com>
Date: Tue, 7 Jan 2025 21:37:46 -0700
Subject: [PATCH 3/5] PR feedback

---
 src/registrar/assets/src/js/getgov/table-base.js    |  4 ++--
 .../assets/src/js/getgov/table-domain-requests.js   | 13 +++++--------
 src/registrar/assets/src/js/getgov/table-domains.js |  6 +-----
 src/registrar/assets/src/js/getgov/table-members.js |  8 ++++----
 4 files changed, 12 insertions(+), 19 deletions(-)

diff --git a/src/registrar/assets/src/js/getgov/table-base.js b/src/registrar/assets/src/js/getgov/table-base.js
index e1d5c11ce..4b7133d64 100644
--- a/src/registrar/assets/src/js/getgov/table-base.js
+++ b/src/registrar/assets/src/js/getgov/table-base.js
@@ -382,7 +382,7 @@ export class BaseTable {
    * for a member, they will also see the kebab column)
    * @param {Object} dataObjects - Data which contains info on domain requests or a user's permission
    * Currently returns a dictionary of either:
-   * - "needsAdditionalColumn": If a new column should be displayed 
+   * - "isDeletable": If a new column should be displayed 
    * - "UserPortfolioPermissionChoices": A user's portfolio permission choices 
    */
   customizeTable(dataObjects){ return {}; }
@@ -406,7 +406,7 @@ export class BaseTable {
    * Returns either: data.members, data.domains or data.domain_requests
    * @param {Object} dataObject - The data used to populate the row content 
    * @param {HTMLElement} tbody - The table body to which the new row is appended to 
-   * @param {Object} customTableOptions - Additional options for customizing row appearance (ie needsAdditionalColumn)
+   * @param {Object} customTableOptions - Additional options for customizing row appearance (ie isDeletable)
    */
   addRow(dataObject, tbody, customTableOptions) {
     throw new Error('addRow must be defined');
diff --git a/src/registrar/assets/src/js/getgov/table-domain-requests.js b/src/registrar/assets/src/js/getgov/table-domain-requests.js
index 3d5d114c3..722ad1eef 100644
--- a/src/registrar/assets/src/js/getgov/table-domain-requests.js
+++ b/src/registrar/assets/src/js/getgov/table-domain-requests.js
@@ -53,7 +53,7 @@ export class DomainRequestsTable extends BaseTable {
     this.toggleExportButton(data.domain_requests);
 
     let needsDeleteColumn = data.domain_requests.some(request => request.is_deletable);
-    return { 'needsAdditionalColumn': needsDeleteColumn };
+    return { 'isDeletable': needsDeleteColumn };
   }
 
   addRow(dataObject, tbody, customTableOptions) {
@@ -71,11 +71,8 @@ export class DomainRequestsTable extends BaseTable {
 
     let markupCreatorRow = '';
     
-    // Forces columns of the domain request and domain tables to align in non-org views
-    let columnWidthLimiterClass = 'width-quarter';
 
     if (this.portfolioValue) {
-      columnWidthLimiterClass = '';
       markupCreatorRow = `
         <td>
             <span class="text-wrap break-word">${request.creator ? request.creator : ''}</span>
@@ -119,15 +116,15 @@ export class DomainRequestsTable extends BaseTable {
       <td data-label="Status">
         ${request.status}
       </td>
-      <td class="${columnWidthLimiterClass}">
-        <div class="grid-row grid-gap">
-          <a href="${actionUrl}">
+      <td class="${ this.portfolioValue ? '' : "width-quarter"}">
+        <div class="tablet:display-flex tablet:flex-row">
+          <a href="${actionUrl}" ${customTableOptions.isDeletable ? "class='margin-right-2'" : ''}>
             <svg class="usa-icon" aria-hidden="true" focusable="false" role="img" width="24">
               <use xlink:href="/public/img/sprite.svg#${request.svg_icon}"></use>
             </svg>
             ${actionLabel} <span class="usa-sr-only">${request.requested_domain ? request.requested_domain : 'New domain request'}</span>
           </a>
-          ${customTableOptions.needsAdditionalColumn ? modalTrigger : ''}
+          ${customTableOptions.isDeletable ? modalTrigger : ''}
         </div>
       </td>
     `;
diff --git a/src/registrar/assets/src/js/getgov/table-domains.js b/src/registrar/assets/src/js/getgov/table-domains.js
index deeb04988..51abf0c6a 100644
--- a/src/registrar/assets/src/js/getgov/table-domains.js
+++ b/src/registrar/assets/src/js/getgov/table-domains.js
@@ -23,12 +23,8 @@ export class DomainsTable extends BaseTable {
     const row = document.createElement('tr');
 
     let markupForSuborganizationRow = '';
-    
-    // Forces columns of the domain request and domain tables to align in non-org views
-    let columnWidthLimiterClass = 'width-quarter';
 
     if (this.portfolioValue) {
-      columnWidthLimiterClass = ''
       markupForSuborganizationRow = `
         <td>
             <span class="text-wrap" aria-label="${domain.suborganization ? suborganization : 'No suborganization'}">${suborganization}</span>
@@ -59,7 +55,7 @@ export class DomainsTable extends BaseTable {
         </svg>
       </td>
       ${markupForSuborganizationRow}
-      <td class="${columnWidthLimiterClass}">
+      <td class="${ this.portfolioValue ? '' : "width-quarter"}">
         <a href="${actionUrl}">
           <svg class="usa-icon" aria-hidden="true" focusable="false" role="img" width="24">
             <use xlink:href="/public/img/sprite.svg#${domain.svg_icon}"></use>
diff --git a/src/registrar/assets/src/js/getgov/table-members.js b/src/registrar/assets/src/js/getgov/table-members.js
index 0b311751d..bc929af4b 100644
--- a/src/registrar/assets/src/js/getgov/table-members.js
+++ b/src/registrar/assets/src/js/getgov/table-members.js
@@ -61,7 +61,7 @@ export class MembersTable extends BaseTable {
       tableHeaderRow.appendChild(extraActionsHeader);
     }
     return { 
-      'needsAdditionalColumn': hasEditPermission,
+      'isDeletable': hasEditPermission,
       'UserPortfolioPermissionChoices' : data.UserPortfolioPermissionChoices
     };
   }
@@ -78,7 +78,7 @@ export class MembersTable extends BaseTable {
     const num_domains = member.domain_urls.length;
     const last_active = this.handleLastActive(member.last_active);
     let cancelInvitationButton = member.type === "invitedmember" ? "Cancel invitation" : "Remove member";
-    const kebabHTML = customTableOptions.needsAdditionalColumn ? generateKebabHTML('remove-member', unique_id, cancelInvitationButton, `for ${member.name}`): ''; 
+    const kebabHTML = customTableOptions.isDeletable ? generateKebabHTML('remove-member', unique_id, cancelInvitationButton, `for ${member.name}`): ''; 
 
     const row = document.createElement('tr');
 
@@ -129,7 +129,7 @@ export class MembersTable extends BaseTable {
           ${member.action_label} <span class="usa-sr-only">${member.name}</span>
         </a>
       </td>
-      ${customTableOptions.needsAdditionalColumn ? '<td>'+kebabHTML+'</td>' : ''}
+      ${customTableOptions.isDeletable ? '<td>'+kebabHTML+'</td>' : ''}
     `;
     tbody.appendChild(row);
     if (domainsHTML || permissionsHTML) {
@@ -137,7 +137,7 @@ export class MembersTable extends BaseTable {
     }
     // This easter egg is only for fixtures that dont have names as we are displaying their emails
     // All prod users will have emails linked to their account
-    if (customTableOptions.needsAdditionalColumn) MembersTable.addMemberModal(num_domains, member.email || "Samwise Gamgee", member_delete_url, unique_id, row);
+    if (customTableOptions.isDeletable) MembersTable.addMemberModal(num_domains, member.email || "Samwise Gamgee", member_delete_url, unique_id, row);
   }
 
   /**

From b27f45b8d782f1042f432f82a4ce618bdb219fcd Mon Sep 17 00:00:00 2001
From: CocoByte <nicolle.leclair@gmail.com>
Date: Wed, 8 Jan 2025 11:00:23 -0700
Subject: [PATCH 4/5] resolved merge conflict

---
 src/registrar/assets/src/js/getgov/table-members.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/registrar/assets/src/js/getgov/table-members.js b/src/registrar/assets/src/js/getgov/table-members.js
index be149a059..6b1a485cb 100644
--- a/src/registrar/assets/src/js/getgov/table-members.js
+++ b/src/registrar/assets/src/js/getgov/table-members.js
@@ -137,7 +137,7 @@ export class MembersTable extends BaseTable {
     }
     // This easter egg is only for fixtures that dont have names as we are displaying their emails
     // All prod users will have emails linked to their account
-    if (customTableOptions.needsAdditionalColumn) MembersTable.addMemberDeleteModal(num_domains, member.email || "Samwise Gamgee", member_delete_url, unique_id, row);
+    if (customTableOptions.isDeletable) MembersTable.addMemberDeleteModal(num_domains, member.email || "Samwise Gamgee", member_delete_url, unique_id, row);
   }
 
   /**

From cad6238932a5d74dc610531e2d7960aa9380294c Mon Sep 17 00:00:00 2001
From: CocoByte <nicolle.leclair@gmail.com>
Date: Fri, 10 Jan 2025 20:58:15 -0700
Subject: [PATCH 5/5] Rename and cleanup

---
 src/registrar/assets/src/js/getgov/table-base.js          | 4 ++--
 .../assets/src/js/getgov/table-domain-requests.js         | 8 ++++----
 src/registrar/assets/src/js/getgov/table-members.js       | 8 ++++----
 .../templates/includes/domain_requests_table.html         | 1 -
 4 files changed, 10 insertions(+), 11 deletions(-)

diff --git a/src/registrar/assets/src/js/getgov/table-base.js b/src/registrar/assets/src/js/getgov/table-base.js
index 4b7133d64..ad57bed01 100644
--- a/src/registrar/assets/src/js/getgov/table-base.js
+++ b/src/registrar/assets/src/js/getgov/table-base.js
@@ -382,7 +382,7 @@ export class BaseTable {
    * for a member, they will also see the kebab column)
    * @param {Object} dataObjects - Data which contains info on domain requests or a user's permission
    * Currently returns a dictionary of either:
-   * - "isDeletable": If a new column should be displayed 
+   * - "hasAdditionalActions": If additional elements need to be added to the Action column 
    * - "UserPortfolioPermissionChoices": A user's portfolio permission choices 
    */
   customizeTable(dataObjects){ return {}; }
@@ -406,7 +406,7 @@ export class BaseTable {
    * Returns either: data.members, data.domains or data.domain_requests
    * @param {Object} dataObject - The data used to populate the row content 
    * @param {HTMLElement} tbody - The table body to which the new row is appended to 
-   * @param {Object} customTableOptions - Additional options for customizing row appearance (ie isDeletable)
+   * @param {Object} customTableOptions - Additional options for customizing row appearance (ie hasAdditionalActions)
    */
   addRow(dataObject, tbody, customTableOptions) {
     throw new Error('addRow must be defined');
diff --git a/src/registrar/assets/src/js/getgov/table-domain-requests.js b/src/registrar/assets/src/js/getgov/table-domain-requests.js
index 722ad1eef..f667a96b5 100644
--- a/src/registrar/assets/src/js/getgov/table-domain-requests.js
+++ b/src/registrar/assets/src/js/getgov/table-domain-requests.js
@@ -52,8 +52,8 @@ export class DomainRequestsTable extends BaseTable {
     // Manage "export as CSV" visibility for domain requests
     this.toggleExportButton(data.domain_requests);
 
-    let needsDeleteColumn = data.domain_requests.some(request => request.is_deletable);
-    return { 'isDeletable': needsDeleteColumn };
+    let isDeletable = data.domain_requests.some(request => request.is_deletable);
+    return { 'hasAdditionalActions': isDeletable };
   }
 
   addRow(dataObject, tbody, customTableOptions) {
@@ -118,13 +118,13 @@ export class DomainRequestsTable extends BaseTable {
       </td>
       <td class="${ this.portfolioValue ? '' : "width-quarter"}">
         <div class="tablet:display-flex tablet:flex-row">
-          <a href="${actionUrl}" ${customTableOptions.isDeletable ? "class='margin-right-2'" : ''}>
+          <a href="${actionUrl}" ${customTableOptions.hasAdditionalActions ? "class='margin-right-2'" : ''}>
             <svg class="usa-icon" aria-hidden="true" focusable="false" role="img" width="24">
               <use xlink:href="/public/img/sprite.svg#${request.svg_icon}"></use>
             </svg>
             ${actionLabel} <span class="usa-sr-only">${request.requested_domain ? request.requested_domain : 'New domain request'}</span>
           </a>
-          ${customTableOptions.isDeletable ? modalTrigger : ''}
+          ${customTableOptions.hasAdditionalActions ? modalTrigger : ''}
         </div>
       </td>
     `;
diff --git a/src/registrar/assets/src/js/getgov/table-members.js b/src/registrar/assets/src/js/getgov/table-members.js
index 6b1a485cb..99a7fc652 100644
--- a/src/registrar/assets/src/js/getgov/table-members.js
+++ b/src/registrar/assets/src/js/getgov/table-members.js
@@ -61,7 +61,7 @@ export class MembersTable extends BaseTable {
       tableHeaderRow.appendChild(extraActionsHeader);
     }
     return { 
-      'isDeletable': hasEditPermission,
+      'hasAdditionalActions': hasEditPermission,
       'UserPortfolioPermissionChoices' : data.UserPortfolioPermissionChoices
     };
   }
@@ -78,7 +78,7 @@ export class MembersTable extends BaseTable {
     const num_domains = member.domain_urls.length;
     const last_active = this.handleLastActive(member.last_active);
     let cancelInvitationButton = member.type === "invitedmember" ? "Cancel invitation" : "Remove member";
-    const kebabHTML = customTableOptions.isDeletable ? generateKebabHTML('remove-member', unique_id, cancelInvitationButton, `for ${member.name}`): ''; 
+    const kebabHTML = customTableOptions.hasAdditionalActions ? generateKebabHTML('remove-member', unique_id, cancelInvitationButton, `for ${member.name}`): ''; 
 
     const row = document.createElement('tr');
 
@@ -129,7 +129,7 @@ export class MembersTable extends BaseTable {
           ${member.action_label} <span class="usa-sr-only">${member.name}</span>
         </a>
       </td>
-      ${customTableOptions.isDeletable ? '<td>'+kebabHTML+'</td>' : ''}
+      ${customTableOptions.hasAdditionalActions ? '<td>'+kebabHTML+'</td>' : ''}
     `;
     tbody.appendChild(row);
     if (domainsHTML || permissionsHTML) {
@@ -137,7 +137,7 @@ export class MembersTable extends BaseTable {
     }
     // This easter egg is only for fixtures that dont have names as we are displaying their emails
     // All prod users will have emails linked to their account
-    if (customTableOptions.isDeletable) MembersTable.addMemberDeleteModal(num_domains, member.email || "Samwise Gamgee", member_delete_url, unique_id, row);
+    if (customTableOptions.hasAdditionalActions) MembersTable.addMemberDeleteModal(num_domains, member.email || "Samwise Gamgee", member_delete_url, unique_id, row);
   }
 
   /**
diff --git a/src/registrar/templates/includes/domain_requests_table.html b/src/registrar/templates/includes/domain_requests_table.html
index f5215f22e..b026a7a6b 100644
--- a/src/registrar/templates/includes/domain_requests_table.html
+++ b/src/registrar/templates/includes/domain_requests_table.html
@@ -187,7 +187,6 @@
                     {% endif %}
                     <th data-sortable="status" scope="col" role="columnheader">Status</th>
                     <th scope="col" role="columnheader">Action</th>
-                    <!-- AJAX will conditionally add a th for delete actions -->
                 </tr>
             </thead>
             <tbody id="domain-requests-tbody">