Remove role=“application” (#548)

gpbl · Nov 15, 2017 · 767346f · 767346f
1 parent f032e3d
commit 767346f
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 5 deletions.
diff --git a/src/DayPicker.js b/src/DayPicker.js
@@ -525,7 +525,6 @@ export default class DayPicker extends Component {
         {...this.props.containerProps}
         className={className}
         ref={el => (this.dayPicker = el)}
-        role="application"
         lang={this.props.locale}
       >
         <div

diff --git a/test/daypicker/rendering.js b/test/daypicker/rendering.js
@@ -100,10 +100,6 @@ describe('DayPicker’s rendering', () => {
     const wrapper = shallow(<DayPicker className="given" />);
     expect(wrapper).toHaveClassName('given');
   });
-  it('should have the application role', () => {
-    const wrapper = shallow(<DayPicker />);
-    expect(wrapper).toHaveProp('role', 'application');
-  });
   it('should use the given tabIndex', () => {
     const wrapper = shallow(<DayPicker tabIndex={-1} />);
     expect(wrapper.find('.DayPicker-wrapper')).toHaveProp('tabIndex', -1);