From 15b9f33c35d3bc4ee172153c1fc963f51a3ccc40 Mon Sep 17 00:00:00 2001
From: Roni Choudhury <roni.choudhury@kitware.com>
Date: Wed, 31 Jan 2024 11:59:34 -0500
Subject: [PATCH 1/3] Process item click as "open item in browser"

---
 web/src/views/FileBrowserView/FileBrowser.vue | 14 +++++++++-----
 1 file changed, 9 insertions(+), 5 deletions(-)

diff --git a/web/src/views/FileBrowserView/FileBrowser.vue b/web/src/views/FileBrowserView/FileBrowser.vue
index 1abf79539..e0a0bb16e 100644
--- a/web/src/views/FileBrowserView/FileBrowser.vue
+++ b/web/src/views/FileBrowserView/FileBrowser.vue
@@ -116,7 +116,7 @@
                 v-for="item in items"
                 :key="item.path"
                 color="primary"
-                @click="selectPath(item)"
+                @click="openItem(item)"
               >
                 <v-icon
                   class="mr-2"
@@ -445,12 +445,16 @@ function locationSlice(index: number) {
   return `${splitLocation.value.slice(0, index + 1).join('/')}/`;
 }
 
-function selectPath(item: AssetPath) {
+function openItem(item: AssetPath) {
   const { asset, path } = item;
 
-  // Return early if path is a file
-  if (asset) { return; }
-  location.value = path;
+  if (asset) {
+    // If the item is an asset, open it in the browser.
+    window.open(inlineURI(asset.asset_id), "_self");
+  } else {
+    // If it's a directory, move into it.
+    location.value = path;
+  }
 }
 
 function navigateToParent() {

From 9d972dad413a348f7f2ccd6d6e1df065224bf694 Mon Sep 17 00:00:00 2001
From: Roni Choudhury <roni.choudhury@kitware.com>
Date: Wed, 31 Jan 2024 12:26:31 -0500
Subject: [PATCH 2/3] Prevent clicks on disabled dropdown from going to item

Without this, clicking on the disabled menu causes the `openItem(...)`
logic to execute, which is not what we want.
---
 web/src/views/FileBrowserView/FileBrowser.vue | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/web/src/views/FileBrowserView/FileBrowser.vue b/web/src/views/FileBrowserView/FileBrowser.vue
index e0a0bb16e..0b80b3739 100644
--- a/web/src/views/FileBrowserView/FileBrowser.vue
+++ b/web/src/views/FileBrowserView/FileBrowser.vue
@@ -116,7 +116,7 @@
                 v-for="item in items"
                 :key="item.path"
                 color="primary"
-                @click="openItem(item)"
+                @click.self="openItem(item)"
               >
                 <v-icon
                   class="mr-2"

From c1279078f52bb12823d973b3cd07fb3493ac6980 Mon Sep 17 00:00:00 2001
From: Roni Choudhury <roni.choudhury@kitware.com>
Date: Wed, 31 Jan 2024 12:27:15 -0500
Subject: [PATCH 3/3] Extend tooltip to educate about clicking on the item

---
 web/src/views/FileBrowserView/FileBrowser.vue | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/web/src/views/FileBrowserView/FileBrowser.vue b/web/src/views/FileBrowserView/FileBrowser.vue
index 0b80b3739..c830f39e1 100644
--- a/web/src/views/FileBrowserView/FileBrowser.vue
+++ b/web/src/views/FileBrowserView/FileBrowser.vue
@@ -158,7 +158,7 @@
                         </v-icon>
                       </v-btn>
                     </template>
-                    <span>Open asset in browser</span>
+                    <span>Open asset in browser (you can also click on the item itself)</span>
                   </v-tooltip>
                 </v-list-item-action>