From 1faec3fb63f27bedea85e153452dde9b95203718 Mon Sep 17 00:00:00 2001
From: Jannik Stehle <jannik.stehle@gmail.com>
Date: Tue, 2 Apr 2024 14:25:35 +0200
Subject: [PATCH] refactor: remove shares panel separation for details query

Removes the shares panel separation between people and link shares when using the `details` query param.
---
 .../components/SideBar/Shares/SharesPanel.vue | 39 +++----------------
 1 file changed, 5 insertions(+), 34 deletions(-)

diff --git a/packages/web-app-files/src/components/SideBar/Shares/SharesPanel.vue b/packages/web-app-files/src/components/SideBar/Shares/SharesPanel.vue
index 3cf1b62cdbc..81e849c711c 100644
--- a/packages/web-app-files/src/components/SideBar/Shares/SharesPanel.vue
+++ b/packages/web-app-files/src/components/SideBar/Shares/SharesPanel.vue
@@ -2,24 +2,19 @@
   <div>
     <oc-loader v-if="sharesLoading" :aria-label="$gettext('Loading list of shares')" />
     <template v-else>
-      <space-members
-        v-if="showSpaceMembers"
-        ref="peopleShares"
-        class="oc-background-highlight oc-p-m oc-mb-s"
-      />
-      <file-shares v-else ref="peopleShares" class="oc-background-highlight oc-p-m oc-mb-s" />
-      <file-links v-if="showLinks" ref="linkShares" class="oc-background-highlight oc-p-m" />
+      <space-members v-if="showSpaceMembers" class="oc-background-highlight oc-p-m oc-mb-s" />
+      <file-shares v-else class="oc-background-highlight oc-p-m oc-mb-s" />
+      <file-links v-if="showLinks" class="oc-background-highlight oc-p-m" />
     </template>
   </div>
 </template>
 
 <script lang="ts">
-import { ComponentPublicInstance, defineComponent, inject, Ref } from 'vue'
+import { defineComponent } from 'vue'
 import FileLinks from './FileLinks.vue'
 import FileShares from './FileShares.vue'
 import SpaceMembers from './SpaceMembers.vue'
 import { useSharesStore } from '@ownclouders/web-pkg'
-import { Resource } from '@ownclouders/web-client'
 import { storeToRefs } from 'pinia'
 
 export default defineComponent({
@@ -39,31 +34,7 @@ export default defineComponent({
     const { loading: sharesLoading } = storeToRefs(sharesStore)
 
     return {
-      sharesLoading,
-      resource: inject<Ref<Resource>>('resource'),
-      activePanel: inject<String>('activePanel')
-    }
-  },
-  watch: {
-    sharesLoading: {
-      handler: function (sharesLoading, old) {
-        // FIXME: !old can be removed as soon as https://github.com/owncloud/web/issues/7621 has been fixed
-        if (!this.activePanel || !old) {
-          return
-        }
-        this.$nextTick(() => {
-          const [panelName, ref] = this.activePanel.split('#')
-
-          if (!ref || !this.$refs[ref]) {
-            return
-          }
-          this.$emit('scrollToElement', {
-            element: (this.$refs[ref] as ComponentPublicInstance).$el,
-            panelName
-          })
-        })
-      },
-      immediate: true
+      sharesLoading
     }
   }
 })