fix: remove navigation role from breadcrumb (#3380)

react-bootstrap · Dec 5, 2018 · 1ac53ad · 1ac53ad
1 parent 8f76314
commit 1ac53ad
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 3 deletions.
diff --git a/src/Breadcrumb.js b/src/Breadcrumb.js
@@ -45,7 +45,6 @@ class Breadcrumb extends React.Component {
     return (
       <Component aria-label={label} className={className} {...props}>
         <ol
-          role="navigation"
           {...listProps}
           className={classNames(bsPrefix, listProps.className)}
         >

diff --git a/test/BreadcrumbSpec.js b/test/BreadcrumbSpec.js
@@ -22,10 +22,10 @@ describe('<Breadcrumb>', () => {
       .should.have.length(1);
   });
 
-  it('Should have a navigation role', () => {
+  it('Should not have a navigation role', () => {
     mount(<Breadcrumb className="custom-one custom-two" />)
       .find('ol[role="navigation"]')
-      .should.have.length(1);
+      .should.have.length(0);
   });
 
   it('Should have an aria-label in ol', () => {