Skip to content

Commit 03791f7

Browse files
feat: persist ownership filter across category changes
Pass allAssets to AssetFilterBar to check for mutable assets across all categories, not just the currently filtered ones. This ensures the ownership filter remains visible when switching categories, providing better UX consistency. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]>
1 parent a3d9747 commit 03791f7

File tree

2 files changed

+5
-3
lines changed

2 files changed

+5
-3
lines changed

src/platform/assets/components/AssetBrowserModal.vue

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -48,6 +48,7 @@
4848
<template #contentFilter>
4949
<AssetFilterBar
5050
:assets="categoryFilteredAssets"
51+
:all-assets="fetchedAssets"
5152
@filter-change="updateFilters"
5253
/>
5354
</template>

src/platform/assets/components/AssetFilterBar.vue

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,7 @@
5454
</div>
5555
</div>
5656
</template>
57-
57+
5858
<script setup lang="ts">
5959
import { computed, ref } from 'vue'
6060
@@ -92,8 +92,9 @@ export interface FilterState {
9292
ownership: OwnershipOption
9393
}
9494
95-
const { assets = [] } = defineProps<{
95+
const { assets = [], allAssets = [] } = defineProps<{
9696
assets?: AssetItem[]
97+
allAssets?: AssetItem[]
9798
}>()
9899
99100
const fileFormats = ref<SelectOption[]>([])
@@ -105,7 +106,7 @@ const { availableFileFormats, availableBaseModels } =
105106
useAssetFilterOptions(assets)
106107
107108
const hasMutableAssets = computed(() =>
108-
assets.some((asset) => asset.is_immutable === false)
109+
allAssets.some((asset) => asset.is_immutable === false)
109110
)
110111
111112
const emit = defineEmits<{

0 commit comments

Comments
 (0)