diff --git a/frontend/src/components/filter/FilterBar.tsx b/frontend/src/components/filter/FilterBar.tsx index adbe736..d220a0a 100644 --- a/frontend/src/components/filter/FilterBar.tsx +++ b/frontend/src/components/filter/FilterBar.tsx @@ -47,14 +47,12 @@ export function FilterBar() { const tagIds = useFilterStore((s) => s.tagIds) const currentSection = useFilterStore((s) => s.currentSection) - // Section-locked pills: each section has a preset filter that defines - // it (Rated → ratingMin, Discarded → flag, Tags → groupBy=tag). The - // matching pill is hidden while you're inside that section because - // toggling it would either be a no-op or break the section. The user - // can still navigate away to change the locked dimension. - const hideRatingPill = currentSection === 'rated' + // Only the Flag pill is hidden inside the Discarded section. Flag has + // exactly two values and the section locks one of them, so the pill + // would only ever toggle the section off — useless. Rating + Tags + // pills stay visible in their sections because the user can refine + // them further (ratingMin >= 3, restrict to specific tag ids). const hideFlagPill = currentSection === 'discarded' - const hideTagsPill = currentSection === 'tags' const setDateFrom = useFilterStore((s) => s.setDateFrom) const setDateTo = useFilterStore((s) => s.setDateTo) @@ -226,40 +224,36 @@ export function FilterBar() { - {/* Rating — hidden in the Rated section since the section already - * pins ratingMin and the only useful tweak (ratingMin >= N) lives - * in the section preset itself. */} - {!hideRatingPill && ( - setRatingMin(0)} - > -
-

Minimum

-
- {[1, 2, 3, 4, 5].map((n) => ( - - ))} -
+ {/* Rating */} + setRatingMin(0)} + > +
+

Minimum

+
+ {[1, 2, 3, 4, 5].map((n) => ( + + ))}
- - )} +
+
{/* Color */} )} - {/* Tags — hidden in the Tags section since the section already - * groups everything by tag and re-applying a tag filter on top - * collapses the view to a single bucket. */} - {!hideTagsPill && allTags.length > 0 && ( + {/* Tags */} + {allTags.length > 0 && (