Skip to content

Commit f09bba4

Browse files
Remove support for breadcrumb back link accessiblePrefix
1 parent 361c276 commit f09bba4

File tree

2 files changed

+16
-27
lines changed

2 files changed

+16
-27
lines changed

src/components/navigation/breadcrumb/Breadcrumb.tsx

Lines changed: 3 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -13,18 +13,12 @@ const Item: Item = ({ className, children, asElement: Element = 'a', ...rest })
1313
</li>
1414
);
1515

16-
type Back = FC<AsElementLink<HTMLAnchorElement> & { accessiblePrefix?: string }>;
16+
type Back = FC<AsElementLink<HTMLAnchorElement>>;
1717

18-
const Back: Back = ({
19-
className,
20-
children,
21-
asElement: Element = 'a',
22-
accessiblePrefix = 'Back to &nbsp;',
23-
...rest
24-
}) => (
18+
const Back: Back = ({ className, children, asElement: Element = 'a', ...rest }) => (
2519
<p className={classNames('nhsuk-breadcrumb__back', className)}>
2620
<Element className="nhsuk-breadcrumb__backlink" {...rest}>
27-
<span className="nhsuk-u-visually-hidden">{accessiblePrefix}</span>
21+
<span className="nhsuk-u-visually-hidden">Back to &nbsp;</span>
2822
{children}
2923
</Element>
3024
</p>

src/components/navigation/breadcrumb/__tests__/Breadcrumb.test.tsx

Lines changed: 13 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -68,24 +68,19 @@ describe('Breadcrumb', () => {
6868
);
6969

7070
describe('The BreadcrumbBack component', () => {
71-
it.each<string | undefined>([undefined, 'Accessible prefix'])(
72-
'Renders as expected with visually hidden text when accessiblePrefix is specified as %s',
73-
(accessiblePrefix) => {
74-
const { container } = render(
75-
<Breadcrumb>
76-
<Breadcrumb.Back href="/back" accessiblePrefix={accessiblePrefix}>
77-
Breadcrumb 2
78-
</Breadcrumb.Back>
79-
</Breadcrumb>,
80-
);
81-
82-
const hiddenSpan = container.querySelector(
83-
'.nhsuk-breadcrumb__backlink > .nhsuk-u-visually-hidden',
84-
);
85-
86-
expect(hiddenSpan?.textContent).toBe(accessiblePrefix ?? 'Back to &nbsp;');
87-
},
88-
);
71+
it('Renders as expected with visually hidden text', () => {
72+
const { container } = render(
73+
<Breadcrumb>
74+
<Breadcrumb.Back href="/back">Breadcrumb 2</Breadcrumb.Back>
75+
</Breadcrumb>,
76+
);
77+
78+
const hiddenSpan = container.querySelector(
79+
'.nhsuk-breadcrumb__backlink > .nhsuk-u-visually-hidden',
80+
);
81+
82+
expect(hiddenSpan?.textContent).toBe('Back to &nbsp;');
83+
});
8984

9085
it.each<ElementType | undefined>([undefined, 'button'])(
9186
'Renders with asElement when specified as %s',

0 commit comments

Comments
 (0)