From f2eca1b5534015db4f16f7401eb58aeebcf357bd Mon Sep 17 00:00:00 2001
From: Dannon Baker <dannon.baker@gmail.com>
Date: Wed, 11 Dec 2024 12:59:54 -0500
Subject: [PATCH 1/2] Fix activity-bar-id handling

---
 client/src/components/ActivityBar/ActivityBar.vue            | 3 +++
 client/src/components/ActivityBar/Items/InteractiveItem.vue  | 5 +++--
 client/src/components/ActivityBar/Items/NotificationItem.vue | 3 ++-
 client/src/components/ActivityBar/Items/UploadItem.vue       | 3 ++-
 4 files changed, 10 insertions(+), 4 deletions(-)

diff --git a/client/src/components/ActivityBar/ActivityBar.vue b/client/src/components/ActivityBar/ActivityBar.vue
index aee942c40ee9..dd2f0bbffc09 100644
--- a/client/src/components/ActivityBar/ActivityBar.vue
+++ b/client/src/components/ActivityBar/ActivityBar.vue
@@ -230,6 +230,7 @@ defineExpose({
                                 v-if="activity.id === 'upload'"
                                 :id="`${activity.id}`"
                                 :key="activity.id"
+                                :activity-bar-id="props.activityBarId"
                                 :icon="activity.icon"
                                 :title="activity.title"
                                 :tooltip="activity.tooltip" />
@@ -237,6 +238,7 @@ defineExpose({
                                 v-else-if="activity.to && activity.id === 'interactivetools'"
                                 :id="`${activity.id}`"
                                 :key="activity.id"
+                                :activity-bar-id="props.activityBarId"
                                 :icon="activity.icon"
                                 :is-active="isActiveRoute(activity.to)"
                                 :title="activity.title"
@@ -274,6 +276,7 @@ defineExpose({
                 <NotificationItem
                     v-if="isConfigLoaded && config.enable_notification_system"
                     id="notifications"
+                    :activity-bar-id="props.activityBarId"
                     :icon="faBell"
                     :is-active="isActiveSideBar('notifications') || isActiveRoute('/user/notifications')"
                     title="Notifications"
diff --git a/client/src/components/ActivityBar/Items/InteractiveItem.vue b/client/src/components/ActivityBar/Items/InteractiveItem.vue
index e9e4756344a9..6b14104c74ab 100644
--- a/client/src/components/ActivityBar/Items/InteractiveItem.vue
+++ b/client/src/components/ActivityBar/Items/InteractiveItem.vue
@@ -17,6 +17,7 @@ export interface Props {
     icon: IconDefinition;
     isActive: boolean;
     to: string;
+    activityBarId: string;
 }
 
 defineProps<Props>();
@@ -27,7 +28,7 @@ const emit = defineEmits<{
 
 const tooltip = computed(() =>
     totalCount.value === 1
-        ? `You currently have 1 active interactive tool`
+        ? "You currently have 1 active interactive tool"
         : `You currently have ${totalCount.value} active interactive tools`
 );
 </script>
@@ -36,7 +37,7 @@ const tooltip = computed(() =>
     <ActivityItem
         v-if="totalCount > 0"
         :id="id"
-        :activity-bar-id="id"
+        :activity-bar-id="activityBarId"
         :icon="icon"
         :indicator="totalCount"
         :is-active="isActive"
diff --git a/client/src/components/ActivityBar/Items/NotificationItem.vue b/client/src/components/ActivityBar/Items/NotificationItem.vue
index 1f1dde9110e3..d72136402dc2 100644
--- a/client/src/components/ActivityBar/Items/NotificationItem.vue
+++ b/client/src/components/ActivityBar/Items/NotificationItem.vue
@@ -11,6 +11,7 @@ const { totalUnreadCount } = storeToRefs(useNotificationsStore());
 
 export interface Props {
     id: string;
+    activityBarId: string;
     title: string;
     icon: IconDefinition;
     isActive: boolean;
@@ -32,7 +33,7 @@ const tooltip = computed(() =>
 <template>
     <ActivityItem
         :id="id"
-        :activity-bar-id="'notifications'"
+        :activity-bar-id="activityBarId"
         :icon="icon"
         :indicator="totalUnreadCount"
         :is-active="isActive"
diff --git a/client/src/components/ActivityBar/Items/UploadItem.vue b/client/src/components/ActivityBar/Items/UploadItem.vue
index 6d5a6df9da64..9b1c5a0f8e11 100644
--- a/client/src/components/ActivityBar/Items/UploadItem.vue
+++ b/client/src/components/ActivityBar/Items/UploadItem.vue
@@ -14,6 +14,7 @@ export interface Props {
     title: string;
     icon: IconDefinition;
     tooltip: string;
+    activityBarId: string;
 }
 
 defineProps<Props>();
@@ -40,7 +41,7 @@ function onUploadModal() {
 <template>
     <ActivityItem
         :id="id"
-        :activity-bar-id="id"
+        :activity-bar-id="activityBarId"
         :title="title"
         :tooltip="tooltip"
         :icon="icon"

From 79f663c8d1543aa19376b21981e42e75ffc0ace8 Mon Sep 17 00:00:00 2001
From: Laila Los <44241786+ElectronicBlueberry@users.noreply.github.com>
Date: Wed, 11 Dec 2024 23:43:08 +0100
Subject: [PATCH 2/2] fix initial value overwrite

---
 client/src/composables/userLocalStorage.ts | 5 ++++-
 client/src/stores/activityStore.ts         | 5 +----
 2 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/client/src/composables/userLocalStorage.ts b/client/src/composables/userLocalStorage.ts
index f0787732a12d..619bbe88bfa0 100644
--- a/client/src/composables/userLocalStorage.ts
+++ b/client/src/composables/userLocalStorage.ts
@@ -13,9 +13,11 @@ import { useHashedUserId } from "./hashedUserId";
 export function useUserLocalStorage<T>(key: string, initialValue: T, user?: Ref<AnyUser>) {
     const { hashedUserId } = useHashedUserId(user);
 
+    let refSyncedRawValue = initialValue;
+
     const storedRef = computed(() => {
         if (hashedUserId.value) {
-            return useLocalStorage(`${key}-${hashedUserId.value}`, initialValue);
+            return useLocalStorage(`${key}-${hashedUserId.value}`, refSyncedRawValue);
         } else {
             return ref(initialValue);
         }
@@ -28,6 +30,7 @@ export function useUserLocalStorage<T>(key: string, initialValue: T, user?: Ref<
         },
         set(newValue) {
             storedRef.value.value = newValue;
+            refSyncedRawValue = newValue as T;
             trigger();
         },
     }));
diff --git a/client/src/stores/activityStore.ts b/client/src/stores/activityStore.ts
index 466ee226bf00..b74a66286f4d 100644
--- a/client/src/stores/activityStore.ts
+++ b/client/src/stores/activityStore.ts
@@ -127,10 +127,7 @@ export const useActivityStore = defineScopedStore("activityStore", (scope) => {
             }
         });
 
-        // update activities stored in local cache only if changes were applied
-        if (JSON.stringify(activities.value) !== JSON.stringify(newActivities)) {
-            activities.value = newActivities;
-        }
+        activities.value = newActivities;
 
         // if toggled side-bar does not exist, choose the first option
         if (toggledSideBar.value !== "") {