Always show search input in CategoryFilterBar
Remove the collapsible search behavior that hid the input behind an icon when ≥4 categories existed. Search should always be visible and match the standard header pattern (w-52 input with icon). Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This commit is contained in:
parent
1e225145c9
commit
05d04c131e
@ -118,7 +118,6 @@ export default function CategoryFilterBar({
|
|||||||
onSearchChange,
|
onSearchChange,
|
||||||
}: CategoryFilterBarProps) {
|
}: CategoryFilterBarProps) {
|
||||||
const [otherOpen, setOtherOpen] = useState(false);
|
const [otherOpen, setOtherOpen] = useState(false);
|
||||||
const [searchCollapsed, setSearchCollapsed] = useState(false);
|
|
||||||
const searchInputRef = useRef<HTMLInputElement>(null);
|
const searchInputRef = useRef<HTMLInputElement>(null);
|
||||||
|
|
||||||
const isAllActive = activeFilters.length === 0;
|
const isAllActive = activeFilters.length === 0;
|
||||||
@ -129,16 +128,6 @@ export default function CategoryFilterBar({
|
|||||||
useSensor(PointerSensor, { activationConstraint: { distance: 5 } }),
|
useSensor(PointerSensor, { activationConstraint: { distance: 5 } }),
|
||||||
);
|
);
|
||||||
|
|
||||||
// Collapse search if there are many categories
|
|
||||||
useEffect(() => {
|
|
||||||
setSearchCollapsed(categories.length >= 4);
|
|
||||||
}, [categories.length]);
|
|
||||||
|
|
||||||
const handleExpandSearch = () => {
|
|
||||||
setSearchCollapsed(false);
|
|
||||||
setTimeout(() => searchInputRef.current?.focus(), 50);
|
|
||||||
};
|
|
||||||
|
|
||||||
const handleDragEnd = (event: DragEndEvent) => {
|
const handleDragEnd = (event: DragEndEvent) => {
|
||||||
const { active, over } = event;
|
const { active, over } = event;
|
||||||
if (!over || active.id === over.id || !onReorderCategories) return;
|
if (!over || active.id === over.id || !onReorderCategories) return;
|
||||||
@ -254,32 +243,18 @@ export default function CategoryFilterBar({
|
|||||||
<div className="flex-1" />
|
<div className="flex-1" />
|
||||||
|
|
||||||
{/* Search */}
|
{/* Search */}
|
||||||
{searchCollapsed ? (
|
<div className="relative shrink-0">
|
||||||
<button
|
<Search className="absolute left-2.5 top-1/2 -translate-y-1/2 h-3.5 w-3.5 text-muted-foreground pointer-events-none" />
|
||||||
type="button"
|
|
||||||
onClick={handleExpandSearch}
|
|
||||||
aria-label="Expand search"
|
|
||||||
className="p-1.5 rounded-md text-muted-foreground hover:text-foreground hover:bg-card-elevated transition-colors duration-150"
|
|
||||||
>
|
|
||||||
<Search className="h-4 w-4" />
|
|
||||||
</button>
|
|
||||||
) : (
|
|
||||||
<div className="relative transition-all duration-200">
|
|
||||||
<Search className="absolute left-2 top-1/2 -translate-y-1/2 h-3.5 w-3.5 text-muted-foreground pointer-events-none" />
|
|
||||||
<Input
|
<Input
|
||||||
ref={searchInputRef}
|
ref={searchInputRef}
|
||||||
type="search"
|
type="search"
|
||||||
placeholder="Search..."
|
placeholder="Search..."
|
||||||
value={searchValue}
|
value={searchValue}
|
||||||
onChange={(e) => onSearchChange(e.target.value)}
|
onChange={(e) => onSearchChange(e.target.value)}
|
||||||
onBlur={() => {
|
className="w-52 h-8 pl-8 text-sm"
|
||||||
if (!searchValue && categories.length >= 4) setSearchCollapsed(true);
|
|
||||||
}}
|
|
||||||
className="w-44 h-8 pl-8 text-sm ring-inset"
|
|
||||||
aria-label="Search"
|
aria-label="Search"
|
||||||
/>
|
/>
|
||||||
</div>
|
</div>
|
||||||
)}
|
|
||||||
</div>
|
</div>
|
||||||
);
|
);
|
||||||
}
|
}
|
||||||
|
|||||||
Loading…
x
Reference in New Issue
Block a user