web app: Fix Recordings download URL problem

As a result of a recent change, the download link in recordings was not supporting right-click to copy the download URL. Now fixed.
MythTV · Jan 22, 2024 · c57f7a8 · c57f7a8
1 parent f3f9d89
commit c57f7a8
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 3 deletions.
diff --git a/mythtv/html/backend/src/app/dashboard/recordings/recordings.component.html b/mythtv/html/backend/src/app/dashboard/recordings/recordings.component.html
@@ -114,7 +114,7 @@ <h2>{{ 'dashboard.recordings.heading' | translate }}</h2>
                         <a href="{{URLencode('/Content/GetRecording?RecordedId=' + program.Recording.RecordedId)}}"
                             target="_blank" pTooltip="{{ 'dashboard.recordings.download' | translate }}"
                             tooltipPosition="left">
-                            <i class="pi pi-download text-primary"></i></a>
+                            <i class="pi pi-download text-primary" [id]="'download_'+ rowIndex"></i></a>
                         <button pButton pRipple icon="pi pi-ellipsis-v" class="p-button-text p-button-primary"
                             (click)="showContextMenu(program,$event)"></button>
                     </td>

diff --git a/mythtv/html/backend/src/app/dashboard/recordings/recordings.component.ts b/mythtv/html/backend/src/app/dashboard/recordings/recordings.component.ts
@@ -212,14 +212,23 @@ export class RecordingsComponent implements OnInit {
     return duration;
   }
 
+  // return true causes default browser right click menu to show
+  // return false suppresses default browser right click menu
   onContextMenu(program: ScheduleOrProgram, event: any) {
+    if (this.selection.length == 0)
+      return true;
+    if (event.target && event.target.id && event.target.id.startsWith('download_'))
+      return true;
     if (this.selection.some((x) => !x)) {
+      // This happens if some entries have not been loaded
       this.sendMessage('error', null, '', this.msg.UndefSelection);
       return false;
     }
-    if (this.selection.some((x) => x.Recording.RecordedId == program.Recording.RecordedId))
+    if (this.selection.some((x) => x.Recording.RecordedId == program.Recording.RecordedId)) {
       this.showContextMenu(null, event);
-    return false;
+      return false;
+    }
+    return true;
   }
 
   onSelectChange() {