From 39dad98d87e42273ecac88c8ca8b26f41d802c63 Mon Sep 17 00:00:00 2001
From: Jin Yi <jin12cc@gmail.com>
Date: Mon, 20 Oct 2025 23:01:56 +0900
Subject: [PATCH] refactor: Extract AssetsSidebarTab template and improve UI
 structure

  - Extract sidebar template into reusable AssetSidebarTemplate component
  - Replace PrimeVue Tabs with TextButton for better visual consistency
  - Add i18n key for "Back to all assets" button
  - Improve job detail view header layout with better spacing
  - Maintain existing functionality while cleaning up template structure
---
 .../sidebar/tabs/AssetSidebarTemplate.vue     | 33 ++++++++
 .../sidebar/tabs/AssetsSidebarTab.vue         | 75 ++++++++++---------
 src/locales/en/main.json                      |  1 +
 3 files changed, 75 insertions(+), 34 deletions(-)
 create mode 100644 src/components/sidebar/tabs/AssetSidebarTemplate.vue

diff --git a/src/components/sidebar/tabs/AssetSidebarTemplate.vue b/src/components/sidebar/tabs/AssetSidebarTemplate.vue
new file mode 100644
index 000000000..94d135ca5
--- /dev/null
+++ b/src/components/sidebar/tabs/AssetSidebarTemplate.vue
@@ -0,0 +1,33 @@
+<template>
+  <div
+    class="flex h-full flex-col bg-white dark-theme:bg-zinc-900"
+    :class="props.class"
+  >
+    <div>
+      <div
+        v-if="slots.top"
+        class="flex min-h-12 items-center border-b border-zinc-200 px-4 py-2 dark-theme:border-zinc-700"
+      >
+        <slot name="top" />
+      </div>
+      <div v-if="slots.header" class="px-4">
+        <slot name="header" />
+      </div>
+    </div>
+    <!-- h-0 to force scrollpanel to grow -->
+    <ScrollPanel class="h-0 grow">
+      <slot name="body" />
+    </ScrollPanel>
+  </div>
+</template>
+
+<script setup lang="ts">
+import ScrollPanel from 'primevue/scrollpanel'
+import { useSlots } from 'vue'
+
+const props = defineProps<{
+  class?: string
+}>()
+
+const slots = useSlots()
+</script>
diff --git a/src/components/sidebar/tabs/AssetsSidebarTab.vue b/src/components/sidebar/tabs/AssetsSidebarTab.vue
index 154b41d75..03df4f658 100644
--- a/src/components/sidebar/tabs/AssetsSidebarTab.vue
+++ b/src/components/sidebar/tabs/AssetsSidebarTab.vue
@@ -1,45 +1,52 @@
 <template>
-  <SidebarTabTemplate
-    :title="isInFolderView ? '' : $t('sideToolbar.mediaAssets')"
-  >
-    <template v-if="isInFolderView" #tool-buttons>
-      <div class="flex w-full items-center gap-2">
-        <span class="font-medium"
-          >Job ID: {{ folderPromptId?.substring(0, 8) }}</span
-        >
-        <button
-          class="rounded p-1 transition-colors hover:bg-neutral-100 dark-theme:hover:bg-neutral-700"
-          :title="$t('g.copy')"
-          @click="copyJobId"
-        >
-          <i class="icon-[lucide--copy] size-4" />
-        </button>
-        <span class="ml-auto text-sm text-neutral-500">
-          {{ formattedExecutionTime }}
-        </span>
+  <AssetsSidebarTemplate>
+    <template #top>
+      <span v-if="!isInFolderView" class="font-bold">
+        {{ $t('sideToolbar.mediaAssets') }}
+      </span>
+      <div v-else class="flex w-full items-center justify-between gap-2">
+        <div class="flex items-center gap-2">
+          <span class="font-bold">{{ $t('Job ID') }}:</span>
+          <span class="text-sm">{{ folderPromptId?.substring(0, 8) }}</span>
+          <i class="mb-1 icon-[lucide--copy] text-sm" @click="copyJobId"></i>
+        </div>
+        <div>
+          <span>{{ formattedExecutionTime }}</span>
+        </div>
       </div>
     </template>
     <template #header>
       <!-- Job Detail View Header -->
-      <div
-        v-if="isInFolderView"
-        class="border-b border-neutral-300 px-4 pt-2 pb-3 dark-theme:border-neutral-700"
-      >
-        <button
-          class="flex items-center gap-2 rounded bg-neutral-100 px-3 py-1.5 text-sm transition-colors hover:bg-neutral-200 dark-theme:bg-neutral-700 dark-theme:hover:bg-neutral-600"
+      <div v-if="isInFolderView" class="pt-4 pb-1">
+        <IconTextButton
+          :label="$t('sideToolbar.backToAssets')"
+          type="secondary"
           @click="exitFolderView"
         >
-          <i class="icon-[lucide--arrow-left] size-4" />
-          <span>Back to all assets</span>
-        </button>
+          <template #icon>
+            <i class="icon-[lucide--arrow-left] size-4" />
+          </template>
+        </IconTextButton>
       </div>
       <!-- Normal Tab View -->
-      <Tabs v-model:value="activeTab" class="w-full">
+      <div v-else class="flex items-center gap-2 pt-4 pb-1">
+        <TextButton
+          :label="$t('sideToolbar.labels.imported')"
+          :type="activeTab === 'input' ? 'secondary' : 'transparent'"
+          @click.stop="activeTab = 'input'"
+        />
+        <TextButton
+          :label="$t('sideToolbar.labels.generated')"
+          :type="activeTab === 'output' ? 'secondary' : 'transparent'"
+          @click.stop="activeTab = 'output'"
+        />
+      </div>
+      <!-- <Tabs v-else v-model:value="activeTab" class="w-full">
         <TabList class="border-b border-neutral-300">
           <Tab value="input">{{ $t('sideToolbar.labels.imported') }}</Tab>
           <Tab value="output">{{ $t('sideToolbar.labels.generated') }}</Tab>
         </TabList>
-      </Tabs>
+      </Tabs> -->
     </template>
     <template #body>
       <VirtualGrid
@@ -87,7 +94,7 @@
         />
       </div>
     </template>
-  </SidebarTabTemplate>
+  </AssetsSidebarTemplate>
   <ResultGallery
     v-model:active-index="galleryActiveIndex"
     :all-gallery-items="galleryItems"
@@ -96,15 +103,13 @@
 
 <script setup lang="ts">
 import ProgressSpinner from 'primevue/progressspinner'
-import Tab from 'primevue/tab'
-import TabList from 'primevue/tablist'
-import Tabs from 'primevue/tabs'
 import { useToast } from 'primevue/usetoast'
 import { computed, onMounted, ref, watch } from 'vue'
 
+import IconTextButton from '@/components/button/IconTextButton.vue'
+import TextButton from '@/components/button/TextButton.vue'
 import NoResultsPlaceholder from '@/components/common/NoResultsPlaceholder.vue'
 import VirtualGrid from '@/components/common/VirtualGrid.vue'
-import SidebarTabTemplate from '@/components/sidebar/tabs/SidebarTabTemplate.vue'
 import ResultGallery from '@/components/sidebar/tabs/queue/ResultGallery.vue'
 import MediaAssetCard from '@/platform/assets/components/MediaAssetCard.vue'
 import { useMediaAssets } from '@/platform/assets/composables/useMediaAssets'
@@ -115,6 +120,8 @@ import {
   getMediaTypeFromFilenamePlural
 } from '@/utils/formatUtil'
 
+import AssetsSidebarTemplate from './AssetSidebarTemplate.vue'
+
 const activeTab = ref<'input' | 'output'>('input')
 const mediaAssets = ref<AssetItem[]>([])
 const selectedAsset = ref<AssetItem | null>(null)
diff --git a/src/locales/en/main.json b/src/locales/en/main.json
index 9125661e2..1a8409f33 100644
--- a/src/locales/en/main.json
+++ b/src/locales/en/main.json
@@ -597,6 +597,7 @@
     "templates": "Templates",
     "assets": "Assets",
     "mediaAssets": "Media Assets",
+    "backToAssets": "Back to all assets",
     "labels": {
       "queue": "Queue",
       "nodes": "Nodes",