From b9f508ae980022521883dba25a0e4af40ce8b22a Mon Sep 17 00:00:00 2001
From: Benedikt Kulmann <benedikt@kulmann.biz>
Date: Fri, 11 Mar 2022 14:50:18 +0100
Subject: [PATCH] Fix reactivity in mobile shares nav

Implicit generated key wasn't incorporating the `active` state and thus
didn't force a re-render when the active state changed.
---
 .../src/components/AppBar/SharesNavigation.vue         | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/packages/web-app-files/src/components/AppBar/SharesNavigation.vue b/packages/web-app-files/src/components/AppBar/SharesNavigation.vue
index 358cd575b84..c2607b67b18 100644
--- a/packages/web-app-files/src/components/AppBar/SharesNavigation.vue
+++ b/packages/web-app-files/src/components/AppBar/SharesNavigation.vue
@@ -19,6 +19,7 @@
         <oc-list>
           <li v-for="navItem in navItems" :key="`shares-navigation-mobile-${navItem.to}`">
             <oc-button
+              :key="`shares-nav-${navItem.to.name}-${navItem.active ? 'active' : 'inactive'}`"
               type="router-link"
               class="oc-my-xs shares-nav-mobile"
               appearance="raw"
@@ -45,8 +46,7 @@ import {
   locationSharesWithMe,
   locationSharesWithOthers
 } from '../../router/shares'
-import { useActiveLocation } from '../../composables'
-import { computed, getCurrentInstance, unref } from '@vue/composition-api'
+import { computed, getCurrentInstance } from '@vue/composition-api'
 import { useRouter } from 'web-pkg/src/composables'
 
 export default {
@@ -66,19 +66,19 @@ export default {
         icon: 'share-forward',
         to: sharesRoutes[locationSharesWithMe.name].path,
         text: $gettext('Shared with me'),
-        active: unref(useActiveLocation(isLocationSharesActive, locationSharesWithMe.name))
+        active: isLocationSharesActive(router, 'files-shares-with-me')
       },
       {
         icon: 'reply',
         to: sharesRoutes[locationSharesWithOthers.name].path,
         text: $gettext('Shared with others'),
-        active: unref(useActiveLocation(isLocationSharesActive, locationSharesWithOthers.name))
+        active: isLocationSharesActive(router, 'files-shares-with-others')
       },
       {
         icon: 'link',
         to: sharesRoutes[locationSharesViaLink.name].path,
         text: $gettext('Shared via link'),
-        active: unref(useActiveLocation(isLocationSharesActive, locationSharesViaLink.name))
+        active: isLocationSharesActive(router, 'files-shares-via-link')
       }
     ])
     return {