Merge pull request #16432 from dannon/ctrlclickdisplay

[23.1] For display (only) allow control+click for new window opening
galaxyproject · Jul 21, 2023 · 4def6e2 · 4def6e2
2 parents 31b20ac + 7f8d4ec
commit 4def6e2
Showing 1 changed file with 12 additions and 2 deletions.
diff --git a/client/src/components/History/Content/ContentOptions.vue b/client/src/components/History/Content/ContentOptions.vue
@@ -11,7 +11,6 @@
             @click.prevent.stop="$emit('showCollectionInfo')">
             <icon icon="info-circle" />
         </b-button>
-
         <!-- Common for all content items -->
         <b-button
             v-if="isDataset"
@@ -22,7 +21,7 @@
             size="sm"
             variant="link"
             :href="displayUrl"
-            @click.prevent.stop="$emit('display')">
+            @click.prevent.stop="onDisplay($event)">
             <icon icon="eye" />
         </b-button>
         <b-button
@@ -121,5 +120,16 @@ export default {
             return this.keyboardSelectable ? "0" : "-1";
         },
     },
+    methods: {
+        onDisplay($event) {
+            // Wrap display handler to allow ctrl/meta click to open in new tab
+            // instead of triggering display.
+            if ($event.ctrlKey || $event.metaKey) {
+                window.open(this.displayUrl, "_blank");
+            } else {
+                this.$emit("display");
+            }
+        },
+    },
 };
 </script>