fix(breadcrumb-item): remove role="presentation" (#2991)

jacobmllr95 · web-flow · commit e84c4a76468e · 2019-04-02T22:05:21.000+02:00
diff --git a/src/components/breadcrumb/breadcrumb-item.js b/src/components/breadcrumb/breadcrumb-item.js
@@ -11,8 +11,7 @@ export default {
       'li',
       mergeData(data, {
         staticClass: 'breadcrumb-item',
-        class: { active: props.active },
-        attrs: { role: 'presentation' }
+        class: { active: props.active }
       }),
       [h(BBreadcrumbLink, { props }, children)]
     )
diff --git a/src/components/breadcrumb/breadcrumb-item.spec.js b/src/components/breadcrumb/breadcrumb-item.spec.js
@@ -8,8 +8,6 @@ describe('breadcrumb-item', () => {
     expect(wrapper.classes()).toContain('breadcrumb-item')
     expect(wrapper.classes()).not.toContain('active')
     expect(wrapper.classes().length).toBe(1)
-    expect(wrapper.attributes('role')).toBeDefined()
-    expect(wrapper.attributes('role')).toBe('presentation')
   })
 
   it('has class active when prop active is set', async () => {
@@ -22,8 +20,6 @@ describe('breadcrumb-item', () => {
     expect(wrapper.classes()).toContain('active')
     expect(wrapper.classes()).toContain('breadcrumb-item')
     expect(wrapper.classes().length).toBe(2)
-    expect(wrapper.attributes('role')).toBeDefined()
-    expect(wrapper.attributes('role')).toBe('presentation')
   })
 
   it('has link as child', async () => {
