fix(datepicker): fix keyboard nav in month/year views

setYear and setMonth return integer values (milliseconds from the epoch). `select` expects a Date object, so convert integers to Dates before passing them to `select`.
mgcrea · Apr 10, 2014 · ef76709 · ef76709
1 parent 82e03ef
commit ef76709
Showing 1 changed file with 8 additions and 8 deletions.
diff --git a/src/datepicker/datepicker.js b/src/datepicker/datepicker.js
@@ -454,10 +454,10 @@ angular.module('mgcrea.ngStrap.datepicker', ['mgcrea.ngStrap.helpers.dateParser'
             },
             onKeyDown: function(evt) {
               var actualMonth = picker.$date.getMonth();
-              if(evt.keyCode === 37) picker.select(picker.$date.setMonth(actualMonth - 1), true);
-              else if(evt.keyCode === 38) picker.select(picker.$date.setMonth(actualMonth - 4), true);
-              else if(evt.keyCode === 39) picker.select(picker.$date.setMonth(actualMonth + 1), true);
-              else if(evt.keyCode === 40) picker.select(picker.$date.setMonth(actualMonth + 4), true);
+              if(evt.keyCode === 37) picker.select(new Date(picker.$date.setMonth(actualMonth - 1)), true);
+              else if(evt.keyCode === 38) picker.select(new Date(picker.$date.setMonth(actualMonth - 4)), true);
+              else if(evt.keyCode === 39) picker.select(new Date(picker.$date.setMonth(actualMonth + 1)), true);
+              else if(evt.keyCode === 40) picker.select(new Date(picker.$date.setMonth(actualMonth + 4)), true);
             }
           }, {
             name: 'year',
@@ -494,10 +494,10 @@ angular.module('mgcrea.ngStrap.datepicker', ['mgcrea.ngStrap.helpers.dateParser'
             },
             onKeyDown: function(evt) {
               var actualYear = picker.$date.getFullYear();
-              if(evt.keyCode === 37) picker.select(picker.$date.setYear(actualYear - 1), true);
-              else if(evt.keyCode === 38) picker.select(picker.$date.setYear(actualYear - 4), true);
-              else if(evt.keyCode === 39) picker.select(picker.$date.setYear(actualYear + 1), true);
-              else if(evt.keyCode === 40) picker.select(picker.$date.setYear(actualYear + 4), true);
+              if(evt.keyCode === 37) picker.select(new Date(picker.$date.setYear(actualYear - 1)), true);
+              else if(evt.keyCode === 38) picker.select(new Date(picker.$date.setYear(actualYear - 4)), true);
+              else if(evt.keyCode === 39) picker.select(new Date(picker.$date.setYear(actualYear + 1)), true);
+              else if(evt.keyCode === 40) picker.select(new Date(picker.$date.setYear(actualYear + 4)), true);
             }
           }];