fix(BreadcrumbItem): Add 'aria-current' attribute (reactstrap#655)

gergely-nagy · Nov 19, 2017 · 644c18e · 644c18e
1 parent 7d5ec13
commit 644c18e
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/src/BreadcrumbItem.js b/src/BreadcrumbItem.js
@@ -29,7 +29,11 @@ const BreadcrumbItem = (props) => {
   ), cssModule);
 
   return (
-    <Tag {...attributes} className={classes} />
+    <Tag
+      {...attributes}
+      className={classes}
+      aria-current={active ? 'page' : undefined}
+    />
   );
 };
 

diff --git a/src/__tests__/BreadcrumbList.spec.js b/src/__tests__/BreadcrumbList.spec.js
@@ -25,12 +25,14 @@ describe('BreadcrumbItem', () => {
     const wrapper = shallow(<BreadcrumbItem>Default BreadcrumbItem</BreadcrumbItem>);
 
     expect(wrapper.hasClass('active')).toBe(false);
+    expect(wrapper.prop('aria-current')).toBe(undefined);
   });
 
   it('should render with the "active" class when the avtive prop is truthy', () => {
     const wrapper = shallow(<BreadcrumbItem active>Default BreadcrumbItem</BreadcrumbItem>);
 
     expect(wrapper.hasClass('active')).toBe(true);
+    expect(wrapper.prop('aria-current')).toBe('page');
   });
 
   it('should render custom tag', () => {