From 3be691306d7a2fc6833bc45c1e58538905ff3c97 Mon Sep 17 00:00:00 2001
From: Jonathan Bout <jonathanbout01@gmail.com>
Date: Thu, 10 Jul 2025 17:37:27 +0200
Subject: [PATCH] cleanup & resolve comment

- remove username check, move current user indicator into v-if=loggedIn
- use the closeHovers method instead of committing to the store
---
 frontend/src/components/Sidebar.vue | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/frontend/src/components/Sidebar.vue b/frontend/src/components/Sidebar.vue
index 3f3c9d35..0ace3cea 100644
--- a/frontend/src/components/Sidebar.vue
+++ b/frontend/src/components/Sidebar.vue
@@ -1,11 +1,11 @@
 <template>
   <div v-show="active" @click="closeHovers" class="overlay"></div>
   <nav :class="{ active }">
-    <button v-if="user.username" @click="toAccountSettings" class="action">
-      <i class="material-icons">person</i>
-      <span>{{ user.username }}</span>
-    </button>
     <template v-if="isLoggedIn">
+      <button @click="toAccountSettings" class="action">
+        <i class="material-icons">person</i>
+        <span>{{ user.username }}</span>
+      </button>
       <button
         class="action"
         @click="toRoot"
@@ -195,7 +195,7 @@ export default {
     },
     toAccountSettings() {
       this.$router.push({ path: "/settings/profile" });
-      this.$store.commit("closeHovers");
+      this.closeHovers();
     },
     toSettings() {
       this.$router.push({ path: "/settings/global" });