Skip to content

Loading…

dateinput bug fix #732

Closed
wants to merge 1 commit into from

2 participants

@bigml

some times 'e' is undefined

@alibby251

Hi bigml,

Do you have a test case where we can see the bug in action please, or a set of steps, which we can follow to replicate the issue?

@bigml

sorry, I can't replicate the issue myself, meet it long times ago. closed the issue.

@bigml bigml closed this
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Apr 9, 2012
  1. @bigml

    undefined bug fix

    bigml committed
This page is out of date. Refresh to see the latest.
Showing with 1 addition and 1 deletion.
  1. +1 −1 src/dateinput/dateinput.js
View
2 src/dateinput/dateinput.js
@@ -300,7 +300,7 @@
e || (e = $.Event("api"));
// focus the input after selection (doesn't work in IE)
- if (e.type == "click" && !$.browser.msie) {
+ if (e && e.type == "click" && !$.browser.msie) {
input.focus();
}
Something went wrong with that request. Please try again.