fix ctrl-click application to open in a new tab (#644)

cmu-sei · Apr 3, 2024 · 842d42e · 842d42e
1 parent 75a52a3
commit 842d42e
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 6 deletions.
diff --git a/src/app/components/player/application-list/application-list.component.html b/src/app/components/player/application-list/application-list.component.html
@@ -22,10 +22,7 @@
           [title]="app.name"
           class="px-0 w-100"
           [href]="app.themedUrl"
-          (click)="
-            $event.preventDefault();
-            app.embeddable ? openInFocusedApp(app) : openInTab(app)
-          "
+          (click)="openApplication(app, $event)"
         >
           <div
             class="d-flex align-items-center"

diff --git a/src/app/components/player/application-list/application-list.component.ts b/src/app/components/player/application-list/application-list.component.ts
@@ -55,8 +55,11 @@ export class ApplicationListComponent implements OnInit, OnChanges, OnDestroy {
   }
 
   // Local Component functions
-  openInTab(app: ApplicationData) {
-    window.open(app.themedUrl, '_blank');
+  openApplication(app: ApplicationData, event: MouseEvent) {
+    if (app.embeddable && !event.ctrlKey) {
+      event.preventDefault();
+      this.openInFocusedApp(app);
+    }
   }
 
   refreshApps() {