PR suggestions

src/registrar/assets/src/js/getgov/table-members.js

@@ -87,7 +87,7 @@ export class MembersTable extends BaseTable {
 
     // generate html blocks for domains and permissions for the member
     let domainsHTML = this.generateDomainsHTML(num_domains, member.domain_names, member.domain_urls, member.action_url);
-    let permissionsHTML = this.generatePermissionsHTML(member.permissions, customTableOptions.UserPortfolioPermissionChoices, unique_id);
+    let permissionsHTML = this.generatePermissionsHTML(member.permissions, customTableOptions.UserPortfolioPermissionChoices);
 
     // domainsHTML block and permissionsHTML block need to be wrapped with hide/show toggle, Expand
     let showMoreButton = '';

src/registrar/assets/src/sass/_theme/_admin.scss

@@ -924,7 +924,3 @@ ul.add-list-reset {
     background-color: transparent !important; 
     }
 }
-
-.dja-detail-table .usa-tag {
-    text-transform: none;
-}

src/registrar/assets/src/sass/_theme/_tables.scss

@@ -41,6 +41,8 @@ th {
   }
 }
 
+// The member table has an extra "expand" row, which looks like a single row.
+// But the DOM disagrees - so we basically need to hide the border on both rows.
 #members__table-wrapper .dotgov-table tr:nth-last-child(2) td,
 #members__table-wrapper .dotgov-table tr:nth-last-child(2) th {
   border-bottom: none;

src/registrar/assets/src/sass/_theme/styles.scss

@@ -26,7 +26,7 @@
 @forward "header";
 @forward "register-form";
 @forward "containers";
-@forward "usa-modal";
+@forward "modals";
 @forward "tags";
 
 /*--------------------------------------------------