Skip to content

Commit 52c86bc

Browse files
fix(breadcrumbs): fix aria-label and onCrumbClick handler
Co-Authored-By: tjiang@box.com <tjiang@box.com>
1 parent 755faf8 commit 52c86bc

File tree

2 files changed

+4
-7
lines changed

2 files changed

+4
-7
lines changed

src/elements/common/breadcrumbs/BreadcrumbDropdown.tsx

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,6 @@ import * as React from 'react';
22
import { DropdownMenu, IconButton } from '@box/blueprint-web';
33
import { Ellipsis } from '@box/blueprint-web-assets/icons/Fill';
44
import type { Crumb } from '../../../common/types/core';
5-
import messages from '../messages';
65

76
export interface BreadcrumbDropdownProps {
87
crumbs: Crumb[];
@@ -12,7 +11,7 @@ export interface BreadcrumbDropdownProps {
1211
const BreadcrumbDropdown = ({ crumbs, onCrumbClick }: BreadcrumbDropdownProps) => (
1312
<DropdownMenu.Root>
1413
<DropdownMenu.Trigger>
15-
<IconButton aria-label={messages.breadcrumbLabel.defaultMessage} icon={Ellipsis} />
14+
<IconButton aria-label="More breadcrumb items" icon={Ellipsis} />
1615
</DropdownMenu.Trigger>
1716
<DropdownMenu.Content>
1817
{crumbs.map(({ id, name }) => (

src/elements/common/breadcrumbs/Breadcrumbs.tsx

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -21,16 +21,14 @@ function filterCrumbs(rootId: string, crumbs: Crumb[]): Crumb[] {
2121
function getBreadcrumb(
2222
crumbs: Crumb | Crumb[],
2323
isLast: boolean,
24-
onCrumbClick: (id: string) => void,
24+
onCrumbClick: (crumb: Crumb) => void,
2525
delimiter: Delimiter,
2626
) {
2727
if (Array.isArray(crumbs)) {
28-
return <BreadcrumbDropdown crumbs={crumbs} onCrumbClick={onCrumbClick} />;
28+
return <BreadcrumbDropdown crumbs={crumbs} onCrumbClick={id => onCrumbClick(crumbs.find(c => c.id === id)!)} />;
2929
}
3030

31-
return (
32-
<Breadcrumb delimiter={delimiter} isLast={isLast} name={crumbs.name} onClick={() => onCrumbClick(crumbs.id)} />
33-
);
31+
return <Breadcrumb delimiter={delimiter} isLast={isLast} name={crumbs.name} onClick={() => onCrumbClick(crumbs)} />;
3432
}
3533

3634
const Breadcrumbs = ({

0 commit comments

Comments
 (0)