From 140524fa4483b9d3319e6cfdd035262f68d06e7a Mon Sep 17 00:00:00 2001
From: Rachid Mrad <rachid.mrad@ecstech.com>
Date: Fri, 1 Nov 2024 14:47:03 -0400
Subject: [PATCH] Standardize accordion positioning for kebobs

---
 src/registrar/assets/js/get-gov.js            | 6 +++---
 src/registrar/models/user.py                  | 1 -
 src/registrar/templates/portfolio_member.html | 7 ++-----
 3 files changed, 5 insertions(+), 9 deletions(-)

diff --git a/src/registrar/assets/js/get-gov.js b/src/registrar/assets/js/get-gov.js
index 185989568..8eb5a8ef7 100644
--- a/src/registrar/assets/js/get-gov.js
+++ b/src/registrar/assets/js/get-gov.js
@@ -202,7 +202,7 @@ function generateKebabHTML(action, unique_id, modal_button_text, screen_reader_t
           </svg>
         </button>
       </div>
-      <div id="more-actions-${unique_id}" class="usa-accordion__content usa-prose shadow-1 left-auto right-0" hidden>
+      <div id="more-actions-${unique_id}" class="usa-accordion__content usa-prose shadow-1 left-auto right-neg-1" hidden>
         <h2>More options</h2>
         ${generateModalButton()} <!-- Desktop button -->
       </div>
@@ -1795,7 +1795,7 @@ class DomainRequestsTable extends BaseTable {
         const delheader = document.createElement('th');
         delheader.setAttribute('scope', 'col');
         delheader.setAttribute('role', 'columnheader');
-        delheader.setAttribute('class', 'delete-header');
+        delheader.setAttribute('class', 'delete-header width-5');
         delheader.innerHTML = `
           <span class="usa-sr-only">Delete Action</span>`;
         let tableHeaderRow = this.tableWrapper.querySelector('thead tr');
@@ -2013,7 +2013,7 @@ class MembersTable extends BaseTable {
       const extraActionsHeader = document.createElement('th');
       extraActionsHeader.setAttribute('id', 'extra-actions');
       extraActionsHeader.setAttribute('role', 'columnheader');
-      extraActionsHeader.setAttribute('class', 'extra-actions-header');
+      extraActionsHeader.setAttribute('class', 'extra-actions-header width-5');
       extraActionsHeader.innerHTML = `
         <span class="usa-sr-only">Extra Actions</span>`;
       let tableHeaderRow = this.tableWrapper.querySelector('thead tr');
diff --git a/src/registrar/models/user.py b/src/registrar/models/user.py
index 741876d98..2d65aa02e 100644
--- a/src/registrar/models/user.py
+++ b/src/registrar/models/user.py
@@ -510,5 +510,4 @@ class User(AbstractUser):
             return True  # The user is the only admin
 
         # If there are other admins or the user is not the only one
-        print(f"{self} is NOT the only admin for portfolio {portfolio}.")
         return False
diff --git a/src/registrar/templates/portfolio_member.html b/src/registrar/templates/portfolio_member.html
index 3b03c09ea..9b6e84c85 100644
--- a/src/registrar/templates/portfolio_member.html
+++ b/src/registrar/templates/portfolio_member.html
@@ -2,9 +2,6 @@
 {% load static field_helpers%}
 
 {% block title %}Organization member 
-<script>
-  console.log("Inline test in the organization member block");
-</script>
 {% endblock %}
 
 {% load static %}
@@ -44,7 +41,7 @@
         data-num-domains="{{ portfolio_permission.get_managed_domains_count }}"
         data-member-email="{{ member.email }}"
       >
-        MEMBER KEBAB SHOULD BE HERE
+        <!-- JS should inject member kebob here -->
         </div>
       {% elif portfolio_invitation %}
         <div id="wrapper-delete-action"
@@ -54,7 +51,7 @@
         data-num-domains="{{ portfolio_invitation.get_managed_domains_count }}"
         data-member-email="{{ portfolio_invitation.email }}"
       >
-        INV KEBAB SHOULD BE HERE
+        <!-- JS should inject invited kebob here -->
       </div>
       {% endif %}
     {% endif %}