From 9b57eb1060c09c9099cb01e01093db9e753fec91 Mon Sep 17 00:00:00 2001
From: Graham Steffaniak <42989099+gtsteffaniak@users.noreply.github.com>
Date: Wed, 18 Oct 2023 20:15:01 -0500
Subject: [PATCH] V0.2.1 - small bugfixes (#58)

Co-authored-by: Graham Steffaniak <graham.steffaniak@autodesk.com>
---
 frontend/src/App.vue                 | 10 +---------
 frontend/src/components/Search.vue   | 25 +++++++++++++++----------
 frontend/src/components/Sidebar.vue  |  2 +-
 frontend/src/views/Layout.vue        |  2 +-
 frontend/src/views/files/Preview.vue |  2 +-
 5 files changed, 19 insertions(+), 22 deletions(-)

diff --git a/frontend/src/App.vue b/frontend/src/App.vue
index 83193bb3..ccfcbe2d 100644
--- a/frontend/src/App.vue
+++ b/frontend/src/App.vue
@@ -9,15 +9,7 @@ __webpack_public_path__ = window.FileBrowser.StaticURL + "/";
 
 export default {
   name: "app",
-  computed: {
-  },
-  mounted() {
-    const loading = document.getElementById("loading");
-    loading.classList.add("done");
-    setTimeout(() => {
-      loading.parentNode.removeChild(loading);
-    }, 200);
-  },
+  computed: {},
 };
 </script>
 
diff --git a/frontend/src/components/Search.vue b/frontend/src/components/Search.vue
index 151bb814..b5a61ede 100644
--- a/frontend/src/components/Search.vue
+++ b/frontend/src/components/Search.vue
@@ -30,7 +30,7 @@
     <div v-if="isMobile && active" id="result" :class="{ hidden: !active }" ref="result">
       <div id="result-list">
         <div class="button" style="width: 100%">
-          Search Context: {{ getContext(this.$route.path) }}
+          Search Context: {{ this.searchContext }}
         </div>
         <ul v-show="results.length > 0">
           <li
@@ -47,7 +47,7 @@
               <i v-else-if="s.archive" class="material-icons archive-icons"> archive </i>
               <i v-else class="material-icons file-icons"> insert_drive_file </i>
               <span class="text-container">
-                {{ basePath(s.path) }}<b>{{ baseName(s.path) }}</b>
+                {{ basePath(s.path,s.dir) }}<b>{{ baseName(s.path) }}</b>
               </span>
             </router-link>
           </li>
@@ -91,7 +91,7 @@
       </div>
     </div>
     <div v-show="!isMobile && active" id="result-desktop" ref="result">
-      <div class="searchContext">Search Context: {{ getContext(this.$route.path) }}</div>
+      <div class="searchContext">Search Context: {{ this.searchContext }}</div>
       <div id="result-list">
         <template>
           <p v-show="isEmpty && isRunning" id="renew">
@@ -178,7 +178,7 @@
               <i v-else-if="s.archive" class="material-icons archive-icons"> archive </i>
               <i v-else class="material-icons file-icons"> insert_drive_file </i>
               <span class="text-container">
-                {{ basePath(s.path) }}<b>{{ baseName(s.path) }}</b>
+                {{ basePath(s.path,s.dir) }}<b>{{ baseName(s.path) }}</b>
               </span>
             </router-link>
           </li>
@@ -535,6 +535,7 @@ export default {
   name: "search",
   data: function () {
     return {
+      searchContext: "./",
       largerThan: "",
       smallerThan: "",
       noneMessage: "Start typing 3 or more characters to begin searching.",
@@ -602,7 +603,7 @@ export default {
     ...mapState(["user", "show"]),
     ...mapGetters(["isListing"]),
     isDarkMode() {
-      return this.user && this.user.darkMode ? this.user.darkMode : darkMode;
+      return this.user && Object.prototype.hasOwnProperty.call(this.user, "darkMode") ? this.user.darkMode : darkMode;
     },
     showBoxes() {
       return this.searchTypes == "";
@@ -634,6 +635,7 @@ export default {
   },
   mounted() {
     window.addEventListener("resize", this.handleResize);
+    this.searchContext = this.getContext(this.$route.path)
     this.handleResize(); // Call this once to set the initial width
   },
   methods: {
@@ -650,15 +652,18 @@ export default {
       let path = "./" + url.substring(url.indexOf("/") + 1);
       return path.replace(/\/+$/, "") + "/";
     },
-    basePath(str) {
-      let parts = str.split("/");
+    basePath(str,isDir) {
+      let parts = str.replace(/(\/$|^\/)/, "").split("/");
       if (parts.length <= 2) {
-        return "/";
+        if (isDir) {
+          return "/"
+        }
+        return "";
       }
       parts.pop();
       parts = parts.join("/") + "/";
-      if (str.endsWith("/")) {
-        parts = "/" + parts;
+      if (isDir) {
+        parts = "/" + parts; // fix weird rtl thing
       }
       return parts;
     },
diff --git a/frontend/src/components/Sidebar.vue b/frontend/src/components/Sidebar.vue
index 20c10823..84c75171 100644
--- a/frontend/src/components/Sidebar.vue
+++ b/frontend/src/components/Sidebar.vue
@@ -129,7 +129,7 @@ export default {
   computed: {
     ...mapState(["user"]),
     isDarkMode() {
-      return this.user && this.user.darkMode ? this.user.darkMode : darkMode;
+      return this.user && Object.prototype.hasOwnProperty.call(this.user, "darkMode") ? this.user.darkMode : darkMode;
     },
     ...mapGetters(["isLogged"]),
     active() {
diff --git a/frontend/src/views/Layout.vue b/frontend/src/views/Layout.vue
index 147d1e75..ba345b5c 100644
--- a/frontend/src/views/Layout.vue
+++ b/frontend/src/views/Layout.vue
@@ -48,7 +48,7 @@ export default {
     ...mapGetters(["isLogged", "progress", "isListing"]),
     ...mapState(["req", "user", "state"]),
     isDarkMode() {
-      return this.user && this.user.darkMode ? this.user.darkMode : darkMode;
+      return this.user && Object.prototype.hasOwnProperty.call(this.user, "darkMode") ? this.user.darkMode : darkMode;
     },
     isExecEnabled: () => enableExec,
     currentView() {
diff --git a/frontend/src/views/files/Preview.vue b/frontend/src/views/files/Preview.vue
index 8ac79582..af7f894d 100644
--- a/frontend/src/views/files/Preview.vue
+++ b/frontend/src/views/files/Preview.vue
@@ -133,7 +133,7 @@ export default {
   computed: {
     ...mapState(["req", "user", "oldReq", "jwt", "loading", "show"]),
     isDarkMode() {
-      return this.user && this.user.darkMode ? this.user.darkMode : darkMode;
+      return this.user && Object.prototype.hasOwnProperty.call(this.user, "darkMode") ? this.user.darkMode : darkMode;
     },
     hasPrevious() {
       return this.previousLink !== "";