From 842d42e51f0f38e2b38692304b35f900e6176eb4 Mon Sep 17 00:00:00 2001
From: Andrew Schlackman <72105194+sei-aschlackman@users.noreply.github.com>
Date: Wed, 3 Apr 2024 12:28:08 -0400
Subject: [PATCH] fix ctrl-click application to open in a new tab (#644)

---
 .../application-list/application-list.component.html       | 5 +----
 .../player/application-list/application-list.component.ts  | 7 +++++--
 2 files changed, 6 insertions(+), 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
index cb66b5f..6bbb45f 100644
--- 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
index 7d3ed45..c4e5d3c 100644
--- 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() {