ros/automaker
1
0
Fork 0
mirror of https://github.com/AutoMaker-Org/automaker.git synced 2026-02-03 08:53:36 +00:00
Code Issues Packages Projects Releases Wiki Activity

refactor: optimize breadcrumb parsing in PathInput component

Browse Source 
- Introduced useMemo for breadcrumb parsing to enhance performance.
- Updated breadcrumb rendering to utilize memoized values for improved efficiency.
This commit is contained in:
Illia Filippov
2025-12-24 19:49:02 +01:00
parent ccc7c6c21d
commit 9dee9fb366
1 changed files with 4 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
apps/ui/src/components/ui/path-input.tsx
View File

@@ -1,5 +1,5 @@
import { useEffect, Fragment, FocusEvent, KeyboardEvent, MouseEvent } from 'react'; import { useEffect, Fragment, FocusEvent, KeyboardEvent, MouseEvent } from 'react';
import { useState, useRef, useCallback } from 'react'; import { useState, useRef, useCallback, useMemo } from 'react';
import { Home, ArrowLeft, Pencil, ArrowRight } from 'lucide-react'; import { Home, ArrowLeft, Pencil, ArrowRight } from 'lucide-react';
import { Button } from '@/components/ui/button'; import { Button } from '@/components/ui/button';
import { Input } from '@/components/ui/input'; import { Input } from '@/components/ui/input';
@@ -168,6 +168,8 @@ function PathInput({
[isEditing] [isEditing]
); );
const breadcrumbs = useMemo(() => parseBreadcrumbs(currentPath), [currentPath]);
const showBreadcrumbs = currentPath && !isEditing && !loading && !error; const showBreadcrumbs = currentPath && !isEditing && !loading && !error;
return ( return (
@@ -217,7 +219,7 @@ function PathInput({
<> <>
<Breadcrumb className="flex-1 min-w-0 overflow-hidden"> <Breadcrumb className="flex-1 min-w-0 overflow-hidden">
<BreadcrumbList className="flex-nowrap overflow-x-auto scrollbar-none"> <BreadcrumbList className="flex-nowrap overflow-x-auto scrollbar-none">
{parseBreadcrumbs(currentPath).map((crumb) => ( {breadcrumbs.map((crumb) => (
<Fragment key={crumb.path}> <Fragment key={crumb.path}>
<BreadcrumbItem className="shrink-0"> <BreadcrumbItem className="shrink-0">
{crumb.isLast ? ( {crumb.isLast ? (