Fix incorrect Sidebar hidden state logic

This commit is contained in:
Gregory Schier
2025-10-28 06:58:31 -07:00
parent b7ad490c9b
commit 9a55426236

View File

@@ -73,8 +73,8 @@ function Sidebar({ className }: { className?: string }) {
const filterRef = useRef<HTMLInputElement>(null); const filterRef = useRef<HTMLInputElement>(null);
const allHidden = useMemo(() => { const allHidden = useMemo(() => {
if (tree?.children?.length === 0) return false; if (tree?.children?.length === 0) return false;
else if (filter) return false; else if (filter) return tree?.children?.every((c) => c.hidden);
else return tree?.children?.every((c) => c.hidden); else return true;
}, [filter, tree?.children]); }, [filter, tree?.children]);
const focusActiveItem = useCallback(() => { const focusActiveItem = useCallback(() => {
@@ -229,7 +229,7 @@ function Sidebar({ className }: { className?: string }) {
)} )}
</div> </div>
{allHidden ? ( {allHidden ? (
<div className="italic text-text-subtle p-3 mt-2 text-sm text-center"> <div className="italic text-text-subtle p-3 text-sm text-center">
No results for <InlineCode>{filter.text}</InlineCode> No results for <InlineCode>{filter.text}</InlineCode>
</div> </div>
) : ( ) : (