Rapidly clicking on DateTextBox in IE 8 throws error pop up message "Can`t move focus to the control because it is invisible... dojo.js"

[kumarann]

Steps to reproduce the error

1) Create a simple html page containing DateTextBox control.

2) Open the page in IE 8.

3) Rapidly click (3-4 clicks) on the date text box.

It throws up an error pop up with message "Can't move focus to the control because it is invisible... dojo.js"

[kumarann]

Have attached a sample html page (hello.html) containing DateTextBox? control to reproduce the issue.

[Douglas Hays]

Hard to recreate but definitely possible (eventually). CalendarLite?._setCurrentFocusAttr contains
newCell.focus();
but that isn't protected by a try/catch. Also I'm not sure if focus should be changed then or should have been deferred until the event handler finishes.

[bill]

possible fix

[bill]

I think it's another problem with timeouts, specifically that:

1. the drop down is opened
2. a focus timer is set
3. the drop down is closed
4. the focus timer fires

It's hard to confirm though since this problem is so intermittent. Could one or both of you try attachment:focusCloseRaceConditionFix.patch​ to see if it fixes the problem for you?

[bill]

Doug -- see above ^^^

[Douglas Hays]

I no longer see the exception after applying the patch

[bill]

In [28454]:

Fix race condition where drop down closes before focus timer fires, fixes #15197 !strict.

[sivakannan]

In the _HasDropDown.js line number 141 contains window.setTimeout(dojo.hitch(dropDown, "focus"), 1); but in the fix, this.defer has been used. It is not working with my code.

[sivakannan]

_HasDropDown.js in dojo 1.6.1

[bill]

this.defer() was added in trunk, for dojo 1.8. Apparently you are trying to use the _HasDropDown.js from trunk against dojo 1.6.1. This is not expected to work.

[mangaya]

This issue occurs in Dojo version 1.7.2, do you have patch or fix for that version too

[bill]

No, but it's trivial to do yourself. Just change the this.defer() to a setTimeout(), and the this._focusDropDownTimer.remove() to a clearTimeout(this._focusDropDownTimer).

Or alternately, even easier, add the defer() method from 1.8 into _WidgetBase.js.