From 3da89ee4e3eafc9d938d15b0a10e45a8bec42162 Mon Sep 17 00:00:00 2001
From: Alex <55413326+AlexRubik@users.noreply.github.com>
Date: Mon, 29 Sep 2025 10:28:55 -0400
Subject: [PATCH] [PM-25616] Refactor app-table-row-scrollable component to use
 openApplication (#16523)

---
 .../all-applications.component.html           | 56 +++++++++----------
 .../all-applications.component.ts             |  6 --
 .../app-table-row-scrollable.component.html   | 28 +++++-----
 .../app-table-row-scrollable.component.ts     |  2 +-
 .../critical-applications.component.html      | 32 +++++------
 .../critical-applications.component.ts        |  6 --
 6 files changed, 60 insertions(+), 70 deletions(-)

diff --git a/bitwarden_license/bit-web/src/app/dirt/access-intelligence/all-applications.component.html b/bitwarden_license/bit-web/src/app/dirt/access-intelligence/all-applications.component.html
index b861925f7cb..febdb5fa0de 100644
--- a/bitwarden_license/bit-web/src/app/dirt/access-intelligence/all-applications.component.html
+++ b/bitwarden_license/bit-web/src/app/dirt/access-intelligence/all-applications.component.html
@@ -64,33 +64,33 @@
         </dirt-card>
       </button>
     </div>
-  }
-  <div class="tw-flex tw-mt-8 tw-mb-4 tw-gap-4">
-    <bit-search
-      [placeholder]="'searchApps' | i18n"
-      class="tw-grow"
-      [formControl]="searchControl"
-    ></bit-search>
-    <button
-      type="button"
-      [buttonType]="'primary'"
-      bitButton
-      [disabled]="!selectedUrls.size"
-      [loading]="markingAsCritical"
-      (click)="markAppsAsCritical()"
-    >
-      <i class="bwi tw-mr-2" [ngClass]="selectedUrls.size ? 'bwi-star-f' : 'bwi-star'"></i>
-      {{ "markAppAsCritical" | i18n }}
-    </button>
-  </div>
+    <div class="tw-flex tw-mt-8 tw-mb-4 tw-gap-4">
+      <bit-search
+        [placeholder]="'searchApps' | i18n"
+        class="tw-grow"
+        [formControl]="searchControl"
+      ></bit-search>
+      <button
+        type="button"
+        [buttonType]="'primary'"
+        bitButton
+        [disabled]="!selectedUrls.size"
+        [loading]="markingAsCritical"
+        (click)="markAppsAsCritical()"
+      >
+        <i class="bwi tw-mr-2" [ngClass]="selectedUrls.size ? 'bwi-star-f' : 'bwi-star'"></i>
+        {{ "markAppAsCritical" | i18n }}
+      </button>
+    </div>
 
-  <app-table-row-scrollable
-    [dataSource]="dataSource"
-    [showRowCheckBox]="true"
-    [showRowMenuForCriticalApps]="false"
-    [selectedUrls]="selectedUrls"
-    [isDrawerIsOpenForThisRecord]="isDrawerOpenForTableRow"
-    [checkboxChange]="onCheckboxChange"
-    [showAppAtRiskMembers]="showAppAtRiskMembers"
-  ></app-table-row-scrollable>
+    <app-table-row-scrollable
+      [dataSource]="dataSource"
+      [showRowCheckBox]="true"
+      [showRowMenuForCriticalApps]="false"
+      [selectedUrls]="selectedUrls"
+      [openApplication]="drawerDetails.invokerId || ''"
+      [checkboxChange]="onCheckboxChange"
+      [showAppAtRiskMembers]="showAppAtRiskMembers"
+    ></app-table-row-scrollable>
+  }
 </div>
diff --git a/bitwarden_license/bit-web/src/app/dirt/access-intelligence/all-applications.component.ts b/bitwarden_license/bit-web/src/app/dirt/access-intelligence/all-applications.component.ts
index 5aff3ed5304..51efafe501d 100644
--- a/bitwarden_license/bit-web/src/app/dirt/access-intelligence/all-applications.component.ts
+++ b/bitwarden_license/bit-web/src/app/dirt/access-intelligence/all-applications.component.ts
@@ -218,10 +218,4 @@ export class AllApplicationsComponent implements OnInit {
       this.selectedUrls.delete(applicationName);
     }
   };
-
-  isDrawerOpenForTableRow = (applicationName: string): boolean => {
-    // Note: This function will be replaced by PR #16523 with openApplication binding
-    // Using private access to BehaviorSubject value for backward compatibility
-    return (this.dataService as any).drawerDetailsSubject?.value?.invokerId === applicationName;
-  };
 }
diff --git a/bitwarden_license/bit-web/src/app/dirt/access-intelligence/app-table-row-scrollable.component.html b/bitwarden_license/bit-web/src/app/dirt/access-intelligence/app-table-row-scrollable.component.html
index 3d3a9baa6e8..ea41dd0aff3 100644
--- a/bitwarden_license/bit-web/src/app/dirt/access-intelligence/app-table-row-scrollable.component.html
+++ b/bitwarden_license/bit-web/src/app/dirt/access-intelligence/app-table-row-scrollable.component.html
@@ -15,7 +15,7 @@
       <td
         bitCell
         *ngIf="showRowCheckBox"
-        [ngClass]="{ 'tw-bg-primary-100': isDrawerIsOpenForThisRecord(row.applicationName) }"
+        [ngClass]="{ 'tw-bg-primary-100': row.applicationName === openApplication }"
         appStopProp
       >
         <input
@@ -30,14 +30,14 @@
       <td
         bitCell
         *ngIf="!showRowCheckBox"
-        [ngClass]="{ 'tw-bg-primary-100': isDrawerIsOpenForThisRecord(row.applicationName) }"
+        [ngClass]="{ 'tw-bg-primary-100': row.applicationName === openApplication }"
       >
         <i class="bwi bwi-star-f" *ngIf="row.isMarkedAsCritical"></i>
       </td>
       <td
         bitCell
         class="tw-cursor-pointer"
-        [ngClass]="{ 'tw-bg-primary-100': isDrawerIsOpenForThisRecord(row.applicationName) }"
+        [ngClass]="{ 'tw-bg-primary-100': row.applicationName === openApplication }"
         (click)="showAppAtRiskMembers(row.applicationName)"
         (keydown.enter)="showAppAtRiskMembers(row.applicationName)"
         (keydown.space)="showAppAtRiskMembers(row.applicationName)"
@@ -49,18 +49,20 @@
       </td>
       <td
         class="tw-cursor-pointer"
+        [ngClass]="{ 'tw-bg-primary-100': row.applicationName === openApplication }"
+        (click)="showAppAtRiskMembers(row.applicationName)"
+        (keydown.enter)="showAppAtRiskMembers(row.applicationName)"
+        (keydown.space)="showAppAtRiskMembers(row.applicationName)"
+        role="button"
+        tabindex="0"
         [attr.aria-label]="'viewItem' | i18n"
-        [ngClass]="{ 'tw-bg-primary-100': isDrawerIsOpenForThisRecord(row.applicationName) }"
-        bitCell
       >
-        <button type="button" tabindex="0" (click)="showAppAtRiskMembers(row.applicationName)">
-          <span>{{ row.applicationName }}</span>
-        </button>
+        <span>{{ row.applicationName }}</span>
       </td>
       <td
         bitCell
         class="tw-cursor-pointer"
-        [ngClass]="{ 'tw-bg-primary-100': isDrawerIsOpenForThisRecord(row.applicationName) }"
+        [ngClass]="{ 'tw-bg-primary-100': row.applicationName === openApplication }"
         (click)="showAppAtRiskMembers(row.applicationName)"
         (keydown.enter)="showAppAtRiskMembers(row.applicationName)"
         (keydown.space)="showAppAtRiskMembers(row.applicationName)"
@@ -75,7 +77,7 @@
       <td
         bitCell
         class="tw-cursor-pointer"
-        [ngClass]="{ 'tw-bg-primary-100': isDrawerIsOpenForThisRecord(row.applicationName) }"
+        [ngClass]="{ 'tw-bg-primary-100': row.applicationName === openApplication }"
         (click)="showAppAtRiskMembers(row.applicationName)"
         (keydown.enter)="showAppAtRiskMembers(row.applicationName)"
         (keydown.space)="showAppAtRiskMembers(row.applicationName)"
@@ -90,7 +92,7 @@
       <td
         bitCell
         class="tw-cursor-pointer"
-        [ngClass]="{ 'tw-bg-primary-100': isDrawerIsOpenForThisRecord(row.applicationName) }"
+        [ngClass]="{ 'tw-bg-primary-100': row.applicationName === openApplication }"
         (click)="showAppAtRiskMembers(row.applicationName)"
         (keydown.enter)="showAppAtRiskMembers(row.applicationName)"
         (keydown.space)="showAppAtRiskMembers(row.applicationName)"
@@ -106,7 +108,7 @@
         bitCell
         data-testid="total-membership"
         class="tw-cursor-pointer"
-        [ngClass]="{ 'tw-bg-primary-100': isDrawerIsOpenForThisRecord(row.applicationName) }"
+        [ngClass]="{ 'tw-bg-primary-100': row.applicationName === openApplication }"
         (click)="showAppAtRiskMembers(row.applicationName)"
         (keydown.enter)="showAppAtRiskMembers(row.applicationName)"
         (keydown.space)="showAppAtRiskMembers(row.applicationName)"
@@ -119,7 +121,7 @@
       <td
         bitCell
         *ngIf="showRowMenuForCriticalApps"
-        [ngClass]="{ 'tw-bg-primary-100': isDrawerIsOpenForThisRecord(row.applicationName) }"
+        [ngClass]="{ 'tw-bg-primary-100': row.applicationName === openApplication }"
         appStopProp
       >
         <button
diff --git a/bitwarden_license/bit-web/src/app/dirt/access-intelligence/app-table-row-scrollable.component.ts b/bitwarden_license/bit-web/src/app/dirt/access-intelligence/app-table-row-scrollable.component.ts
index e9574413825..01f3b8fb494 100644
--- a/bitwarden_license/bit-web/src/app/dirt/access-intelligence/app-table-row-scrollable.component.ts
+++ b/bitwarden_license/bit-web/src/app/dirt/access-intelligence/app-table-row-scrollable.component.ts
@@ -18,7 +18,7 @@ export class AppTableRowScrollableComponent {
   @Input() showRowMenuForCriticalApps: boolean = false;
   @Input() showRowCheckBox: boolean = false;
   @Input() selectedUrls: Set<string> = new Set<string>();
-  @Input() isDrawerIsOpenForThisRecord!: (applicationName: string) => boolean;
+  @Input() openApplication: string = "";
   @Input() showAppAtRiskMembers!: (applicationName: string) => void;
   @Input() unmarkAsCritical!: (applicationName: string) => void;
   @Input() checkboxChange!: (applicationName: string, $event: Event) => void;
diff --git a/bitwarden_license/bit-web/src/app/dirt/access-intelligence/critical-applications.component.html b/bitwarden_license/bit-web/src/app/dirt/access-intelligence/critical-applications.component.html
index 18b114515af..ed2a6b96524 100644
--- a/bitwarden_license/bit-web/src/app/dirt/access-intelligence/critical-applications.component.html
+++ b/bitwarden_license/bit-web/src/app/dirt/access-intelligence/critical-applications.component.html
@@ -82,21 +82,21 @@
         </dirt-card>
       </button>
     </div>
-  }
-  <div class="tw-flex tw-mt-8 tw-mb-4 tw-gap-4">
-    <bit-search
-      [placeholder]="'searchApps' | i18n"
-      class="tw-grow"
-      [formControl]="searchControl"
-    ></bit-search>
-  </div>
+    <div class="tw-flex tw-mt-8 tw-mb-4 tw-gap-4">
+      <bit-search
+        [placeholder]="'searchApps' | i18n"
+        class="tw-grow"
+        [formControl]="searchControl"
+      ></bit-search>
+    </div>
 
-  <app-table-row-scrollable
-    [dataSource]="dataSource"
-    [showRowCheckBox]="false"
-    [showRowMenuForCriticalApps]="true"
-    [isDrawerIsOpenForThisRecord]="isDrawerOpenForTableRow"
-    [showAppAtRiskMembers]="showAppAtRiskMembers"
-    [unmarkAsCritical]="unmarkAsCritical"
-  ></app-table-row-scrollable>
+    <app-table-row-scrollable
+      [dataSource]="dataSource"
+      [showRowCheckBox]="false"
+      [showRowMenuForCriticalApps]="true"
+      [openApplication]="drawerDetails.invokerId || ''"
+      [showAppAtRiskMembers]="showAppAtRiskMembers"
+      [unmarkAsCritical]="unmarkAsCritical"
+    ></app-table-row-scrollable>
+  }
 </div>
diff --git a/bitwarden_license/bit-web/src/app/dirt/access-intelligence/critical-applications.component.ts b/bitwarden_license/bit-web/src/app/dirt/access-intelligence/critical-applications.component.ts
index 5cea6903568..9110779c980 100644
--- a/bitwarden_license/bit-web/src/app/dirt/access-intelligence/critical-applications.component.ts
+++ b/bitwarden_license/bit-web/src/app/dirt/access-intelligence/critical-applications.component.ts
@@ -201,10 +201,4 @@ export class CriticalApplicationsComponent implements OnInit {
     const data = this.reportService.generateAtRiskApplicationList(this.dataSource.data);
     this.dataService.setDrawerForOrgAtRiskApps(data, invokerId);
   };
-
-  isDrawerOpenForTableRow = (applicationName: string) => {
-    // Note: This function will be replaced by PR #16523 with openApplication binding
-    // Using private access to BehaviorSubject value for backward compatibility
-    return (this.dataService as any).drawerDetailsSubject?.value?.invokerId === applicationName;
-  };
 }