Merge pull request #79 from pacien/oz-viewer

Oz viewer - code review fixes

5 files changed, 11 insertions, 9 deletions

diff --git a/viewer/src/assets/scss/theme.scss b/viewer/src/assets/scss/theme.scss
index 1b474e8..f46b9f5 100644
--- a/viewer/src/assets/scss/theme.scss
+++ b/viewer/src/assets/scss/theme.scss
@@ -33,7 +33,7 @@ $loader-color: #272727;
 // Overrides - Buefy
 $link: $panel-top-txtcolor;
 $link-hover: lightblue;
-$disabled-color: rgba($link, 0.3);
+$disabled-color: #656589;
 $control-radius: 0;
 $input-radius: 0;
 $loading-background: $panel-top-bgcolor;
diff --git a/viewer/src/components/LdBreadcrumb.vue b/viewer/src/components/LdBreadcrumb.vue
index 8c5b7eb..171e96a 100644
--- a/viewer/src/components/LdBreadcrumb.vue
+++ b/viewer/src/components/LdBreadcrumb.vue
@@ -42,8 +42,10 @@ export default class LdBreadcrumb extends Vue {
 </script>
 
 <style lang="scss">
+@import "@/assets/scss/theme.scss";
+
 .ld-breadcrumb {
-  border-left: 2px solid rgba(white, 0.1);
+  border-left: 1px solid $disabled-color;
   padding-left: 15px;
   display: flex;
   list-style: none;
diff --git a/viewer/src/components/LdCommand.vue b/viewer/src/components/LdCommand.vue
index 07cb354..8c6ccfc 100644
--- a/viewer/src/components/LdCommand.vue
+++ b/viewer/src/components/LdCommand.vue
@@ -19,16 +19,16 @@
 
 <template>
   <div class="flex command-btns">
-    <div class="link" title="$t('title.tags')" @click="$uiStore.toggleFullWidth()">
+    <div class="link" :title="$t('title.tags')" @click="$uiStore.toggleFullWidth()">
       <fa-icon :icon="commandTagsIcon()" size="lg" />
     </div>
-    <router-link to="/" :class="{'disabled': isRoot()}" title="$t('title.home')">
+    <router-link to="/" :class="{'disabled': isRoot()}" :title="$t('title.home')">
       <fa-icon icon="home" size="lg" />
     </router-link>
-    <div class="link" title="$t('title.back')" @click="$router.go(-1)">
+    <div class="link" :title="$t('title.back')" @click="$router.go(-1)">
       <fa-icon icon="arrow-left" size="lg" />
     </div>
-    <router-link :class="{'disabled': isRoot()}" title="$t('title.parent')" :to="parent()">
+    <router-link :class="{'disabled': isRoot()}" :title="$t('title.parent')" :to="parent()">
       <fa-icon icon="folder" size="xs" />
       <fa-icon icon="level-up-alt" size="lg" />
     </router-link>
diff --git a/viewer/src/plugins/fontawesome.ts b/viewer/src/plugins/fontawesome.ts
index a15f4ce..d70746d 100644
--- a/viewer/src/plugins/fontawesome.ts
+++ b/viewer/src/plugins/fontawesome.ts
@@ -33,7 +33,7 @@ import {
     faTags,
     faAngleRight,
     faWindowClose,
-    faQuestionCircle,
+    faFile,
 } from "@fortawesome/free-solid-svg-icons";
 
 library.add(
@@ -48,7 +48,7 @@ library.add(
     faTags,
     faAngleRight,
     faWindowClose,
-    faQuestionCircle,
+    faFile,
 );
 
 config.autoAddCss = false;
diff --git a/viewer/src/tools.ts b/viewer/src/tools.ts
index 94fdf33..124e183 100644
--- a/viewer/src/tools.ts
+++ b/viewer/src/tools.ts
@@ -52,7 +52,7 @@ export default class Tools {
         return "folder";
       case "other":
       default:
-        return "question-circle";
+        return "file";
     }
   }