From fa0ffd890d5e4fee962102dcb7443297822fb339 Mon Sep 17 00:00:00 2001
From: Grady Ward <grady.b.ward@gmail.com>
Date: Sat, 20 Jan 2024 19:25:33 -0700
Subject: [PATCH] Improves Nav Organization

---
 frontend/components/standard/Avatar.vue |  20 ++++
 frontend/components/standard/Nav.vue    | 129 +++++++++++++++++++++---
 frontend/lang/en.json                   |   4 +-
 frontend/plugins/primevue.ts            |   2 +
 4 files changed, 140 insertions(+), 15 deletions(-)
 create mode 100644 frontend/components/standard/Avatar.vue

diff --git a/frontend/components/standard/Avatar.vue b/frontend/components/standard/Avatar.vue
new file mode 100644
index 0000000..b5f8623
--- /dev/null
+++ b/frontend/components/standard/Avatar.vue
@@ -0,0 +1,20 @@
+<script setup lang="ts">
+const props = defineProps<{
+  name?: string | undefined
+}>()
+
+const singleCharacter = computed(() => {
+  return props.name?.[0] || 'U'
+})
+</script>
+
+<template>
+  <div
+    class="flex align-items-center justify-content-center border-circle bg-teal-400"
+    style="height: 2.5rem; width: 2.5rem;"
+  >
+    <span class="text-2xl text-white font-bold">
+      {{ singleCharacter }}
+    </span>
+  </div>
+</template>
diff --git a/frontend/components/standard/Nav.vue b/frontend/components/standard/Nav.vue
index cb2d0e8..00d5bae 100644
--- a/frontend/components/standard/Nav.vue
+++ b/frontend/components/standard/Nav.vue
@@ -1,5 +1,6 @@
 <script setup lang="ts">
 import { type MenuItem } from 'primevue/menuitem'
+import type * as PrimeVueMenu from 'primevue/menu'
 
 const { t } = useI18n()
 const localePath = useLocalePath()
@@ -15,6 +16,14 @@ const { isAdmin, maybeMe } = await getMaybeMe()
 const prefix = 'components/standard/Nav'
 const tt = (s: string) => t(`${prefix}.${s}`)
 const menuHidden = useState<boolean>(`${prefix}.menuHidden`, () => false)
+const userMenu = useState<{ toggle: (e: Event) => void } | null>(`${prefix}.userMenu`, () => null)
+const userMenuVisible = useState<boolean>(`${prefix}.userMenuVisible`, () => false)
+
+const toggleUserMenu = (e: Event) => {
+  if (userMenu.value) {
+    userMenu.value.toggle(e)
+  }
+}
 
 const menuStyles = computed(() => {
   return {
@@ -44,17 +53,11 @@ const menuItems = computed(() => {
       to: localePath('/admin'),
     })
   }
-  if (maybeMe.value) {
+  if (isAuthenticated.value) {
     result.push({
       label: tt('My Data'),
       to: localePath('/my-data'),
     })
-  }
-  if (isAuthenticated.value) {
-    result.push({
-      label: tt('Sign Out'),
-      command: () => { void signOut() },
-    })
   } else {
     result.push({
       label: tt('Sign In'),
@@ -63,6 +66,26 @@ const menuItems = computed(() => {
   }
   return result
 })
+const userMenuItems = computed(() => {
+  const result: MenuItem[] = [{
+    label: tt('Account'),
+    icon: 'pi pi-cog',
+    to: localePath('/user/me'),
+  }, {
+    label: tt('My Data'),
+    icon: 'pi pi-list',
+    to: localePath('/my-data'),
+  }, {
+    label: tt('Audit Logs'),
+    icon: 'pi pi-lock',
+    to: localePath('/audit-logs'),
+  }, {
+    label: tt('Sign Out'),
+    icon: 'pi pi-sign-out',
+    command: () => { void signOut() },
+  }]
+  return result
+})
 </script>
 
 <template>
@@ -74,15 +97,26 @@ const menuItems = computed(() => {
         class="h-3rem pb-2"
         @click="() => router.push(localePath('/'))"
       />
-      <PVButton
-        :icon="menuHidden ? 'pi pi-bars' : 'pi pi-times'"
-        :class="menuHidden ? 'p-button-text' : 'border-bottom-noround p-button-primary'"
-        class="sm:hidden p-button-lg h-3rem"
-        @click="() => menuHidden = !menuHidden"
-      />
+      <div class="flex gap-1 align-items-center">
+        <PVButton
+          v-show="maybeMe !== undefined"
+          v-tooltip="tt('Settings')"
+          icon="pi pi-user"
+          class="sm:hidden ml-2 flex-shrink-0"
+          rounded
+          :class="userMenuVisible ? 'p-button-primary' : 'p-button-text'"
+          @click="toggleUserMenu"
+        />
+        <PVButton
+          :icon="menuHidden ? 'pi pi-bars' : 'pi pi-times'"
+          :class="menuHidden ? 'p-button-text' : 'border-bottom-noround p-button-primary'"
+          class="sm:hidden p-button-lg h-3rem"
+          @click="() => menuHidden = !menuHidden"
+        />
+      </div>
     </div>
     <div
-      class="flex gap-2 sm:p-1 flex-1 flex-column sm:flex-row border-primary sm:border-none sm:max-h-full border-round justify-content-end"
+      class="flex gap-2 sm:p-1 flex-1 flex-column sm:flex-row border-primary sm:border-none sm:max-h-full border-round justify-content-end align-items-center"
       :style="menuStyles"
     >
       <template
@@ -93,6 +127,7 @@ const menuItems = computed(() => {
           :key="index"
           :class="mi.to === router.currentRoute.value.fullPath ? 'border-noround sm:border-round' : 'p-button-text'"
           :to="mi.to"
+          :external="mi.external"
           :label="`${mi.label}`"
         />
         <PVButton
@@ -103,6 +138,72 @@ const menuItems = computed(() => {
           @click="mi.command"
         />
       </template>
+      <PVButton
+        v-show="maybeMe !== undefined"
+        v-tooltip="tt('Settings')"
+        icon="pi pi-user"
+        class="hidden sm:flex ml-2 flex-shrink-0"
+        rounded
+        :class="userMenuVisible ? 'p-button-primary' : 'p-button-outlined'"
+        @click="toggleUserMenu"
+      />
+      <PVOverlayPanel
+        ref="userMenu"
+        :pt="{
+          content: 'p-0',
+        }"
+        class="caret-primary"
+        @hide="() => userMenuVisible = false"
+        @show="() => userMenuVisible = true"
+      >
+        <div class="bg-primary p-3">
+          <div class="flex gap-3 align-items-center">
+            <StandardAvatar :name="maybeMe?.name" />
+            <div class="flex flex-column gap-1">
+              <span class="font-bold text-lg text-white">
+                {{ maybeMe?.name }}
+              </span>
+              <span class="text-sm text-white">
+                {{ maybeMe?.enteredEmail }}
+              </span>
+            </div>
+          </div>
+        </div>
+        <div
+          class="flex flex-column gap-1 p-2"
+          @click="toggleUserMenu"
+        >
+          <template
+            v-for="(mi, index) in userMenuItems"
+          >
+            <LinkButton
+              v-if="mi.to"
+              :key="index"
+              :class="mi.to === router.currentRoute.value.fullPath ? 'border-noround sm:border-round' : 'p-button-text'"
+              :to="mi.to"
+              :external="mi.external"
+              :icon="mi.icon"
+              :label="`${mi.label}`"
+            />
+            <PVButton
+              v-else
+              :key="mi.label"
+              :label="mi.label"
+              :icon="mi.icon"
+              class="p-button-text"
+              @click="mi.command"
+            />
+          </template>
+        </div>
+      </PVOverlayPanel>
     </div>
   </div>
 </template>
+
+<style lang="scss">
+.caret-primary.p-overlaypanel{
+  &::before, &::after {
+    border-bottom-color: var(--primary-color)
+  }
+}
+</style>
diff --git a/frontend/lang/en.json b/frontend/lang/en.json
index 9feec45..982f5ab 100644
--- a/frontend/lang/en.json
+++ b/frontend/lang/en.json
@@ -267,7 +267,9 @@
     "Home": "Portfolio Analysis Tool",
     "My Data": "My Data",
     "Sign In": "Sign In",
-    "Sign Out": "Sign Out"
+    "Sign Out": "Sign Out",
+    "Account": "Account",
+    "Audit Logs": "Audit Logs"
   },
   "components/user/Editor": {
     "The name that will be associated with": "The name that will be associated with ",
diff --git a/frontend/plugins/primevue.ts b/frontend/plugins/primevue.ts
index 6353731..bb77250 100644
--- a/frontend/plugins/primevue.ts
+++ b/frontend/plugins/primevue.ts
@@ -22,6 +22,7 @@ import Textarea from 'primevue/textarea'
 import Tooltip from 'primevue/tooltip'
 import TriStateCheckbox from 'primevue/tristatecheckbox'
 import Message from 'primevue/message'
+import Menu from 'primevue/menu'
 import MultiSelect from 'primevue/multiselect'
 import OverlayPanel from 'primevue/overlaypanel'
 import ProgressSpinner from 'primevue/progressspinner'
@@ -48,6 +49,7 @@ export default defineNuxtPlugin(({ vueApp }) => {
   vueApp.component('PVInputText', InputText)
   vueApp.component('PVInputSwitch', InputSwitch)
   vueApp.component('PVMessage', Message)
+  vueApp.component('PVMenu', Menu)
   vueApp.component('PVMultiSelect', MultiSelect)
   vueApp.component('PVOverlayPanel', OverlayPanel)
   vueApp.component('PVProgressSpinner', ProgressSpinner)