Merge pull request #646 from gpbl/fix-630

Fix error with focusing outside nodes

Author: gpbl

src/DayPicker.js

@@ -300,7 +300,7 @@ export default class DayPicker extends Component {
   focusPreviousDay(dayNode) {
     const dayNodes = Helpers.getDayNodes(this.dayPicker, this.props.classNames);
     const dayNodeIndex = Helpers.nodeListToArray(dayNodes).indexOf(dayNode);
-
+    if (dayNodeIndex === -1) return;
     if (dayNodeIndex === 0) {
       this.showPreviousMonth(() => this.focusLastDayOfMonth());
     } else {
@@ -311,7 +311,7 @@ export default class DayPicker extends Component {
   focusNextDay(dayNode) {
     const dayNodes = Helpers.getDayNodes(this.dayPicker, this.props.classNames);
     const dayNodeIndex = Helpers.nodeListToArray(dayNodes).indexOf(dayNode);
-
+    if (dayNodeIndex === -1) return;
     if (dayNodeIndex === dayNodes.length - 1) {
       this.showNextMonth(() => this.focusFirstDayOfMonth());
     } else {

test/daypicker/methods.js

@@ -258,6 +258,11 @@ describe('DayPicker’s methods', () => {
         expect(document.activeElement.innerHTML).toBe('31');
         expect(instance.state.currentMonth.getMonth()).toBe(4);
       });
+      it('should not throw an error when the node is not found', () => {
+        expect(() =>
+          instance.focusPreviousDay(document.createElement('div'))
+        ).not.toThrow();
+      });
     });
 
     describe('focusNextDay()', () => {
@@ -301,6 +306,11 @@ describe('DayPicker’s methods', () => {
         expect(document.activeElement.innerHTML).toBe('1');
         expect(instance.state.currentMonth.getMonth()).toBe(2);
       });
+      it('should not throw an error when the node is not found', () => {
+        expect(() =>
+          instance.focusNextDay(document.createElement('div'))
+        ).not.toThrow();
+      });
     });
 
     describe('focusNextWeek()', () => {