Skip to content

Commit

Permalink
Merge pull request ExactTarget#338 from vernak2539/remove_input_parsing
Browse files Browse the repository at this point in the history
Removing date parsing on user input because of FF
  • Loading branch information
vernak2539 committed Nov 21, 2013
2 parents b5a85c5 + 3d0139e commit ee48352
Showing 1 changed file with 7 additions and 3 deletions.
10 changes: 7 additions & 3 deletions src/datepicker.js
Original file line number Diff line number Diff line change
Expand Up @@ -713,7 +713,9 @@ define(function (require) {
}
},

_keyupDateUpdate: function( e ) {
// removed this because of an issue with date parsing in Firefox (really wish all the apis were the same)
// will be put back in 2.6 using moment.js
/*_keyupDateUpdate: function( e ) {
var validLength = this.formatDate( this.date ).length;
var inputValue = this.$input.val();
Expand All @@ -737,7 +739,7 @@ define(function (require) {
} else {
return false;
}
},
},*/

_initializeCalendarElements: function() {
this.$input = this.$element.find( 'input[type="text"]' );
Expand All @@ -755,7 +757,9 @@ define(function (require) {
},

_addBindings: function() {
this.$input.on( 'keyup', $.proxy( this._keyupDateUpdate, this ) );
// removed this because of an issue with date parsing in Firefox (really wish all the apis were the same)
// will be put back in 2.6 using moment.js
//this.$input.on( 'keyup', $.proxy( this._keyupDateUpdate, this ) );
this.$calendar.on( 'click', $.proxy( this._emptySpace, this) );

this.$header.find( '.left' ).on( 'click', $.proxy( this._previous, this ) );
Expand Down

0 comments on commit ee48352

Please sign in to comment.