From 93ee141d03cbd6f3a9cb75e5a6df2073170bde5b Mon Sep 17 00:00:00 2001 From: Glen Padua Date: Wed, 25 Sep 2019 22:23:19 +0530 Subject: [PATCH] feat(NavbarToggler): add aria-label to NavbarToggler This adds the aria-label to the NavbarToggler as seen in the bootstrap docs. Fixes #1632 --- src/NavbarToggler.js | 2 +- src/__tests__/NavbarToggler.spec.js | 3 ++- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/src/NavbarToggler.js b/src/NavbarToggler.js index 65c6fa9be..a0a892c9c 100644 --- a/src/NavbarToggler.js +++ b/src/NavbarToggler.js @@ -31,7 +31,7 @@ const NavbarToggler = (props) => { ), cssModule); return ( - + {children || } ); diff --git a/src/__tests__/NavbarToggler.spec.js b/src/__tests__/NavbarToggler.spec.js index eded23b5d..586c728e2 100644 --- a/src/__tests__/NavbarToggler.spec.js +++ b/src/__tests__/NavbarToggler.spec.js @@ -5,7 +5,8 @@ import { NavbarToggler } from '../'; describe('NavbarToggler', () => { it('should render .navbar-toggler markup', () => { const wrapper = shallow(); - + + expect(wrapper.prop('aria-label')).toBe('Toggle navigation'); expect(wrapper.html()).toBe(''); });