Change configuration during postbacks so that 'onfocus' callbacks are not wired. Rewire the 'onfocus' callbacks with the newly DOM attached elements after a delay so that ice.applyFocus command has a chance to run.

The applied fix is replacing the wiring of the previous onfocus callbacks that pop up the calendar with callbacks that run after the updates are applied including the ice.applyFocus evaluation. Also the callbacks will be wired to the new elements that updated the dom to make sure they properly respond to the 'focus' events.

Assignee: yip.ng (was: Mircea Toma)
Yip, pls review Mircea's commit to see if needs to be refactored or cleaned up.
(Restricted to icesoft-internal-developers group)

1. Still doesn't seem to work on the test app even after changing the timer from 500 to 5000. Focus still just disappears after selecting a date.
2. After selecting a date, tabbing into input field doesn't popup calendar anymore.

Yip, lets revert Mircea's commit for the Beta release and he can take another crack at this later.

Assigned to Mircea to devise a new fix that avoids the negative side-effects that the original commit had.

Modified ice.ace.Calendar.prototype.bindDateSelectListener to set the focus back on the input element after the update is applied. The $.datepicker._showDatepicker callback is unbound, focus applied and then the callback is bound again.

[2:49:00 PM] Ken Fyten: Yip please verify Mircea's fix.

Still doesn't seem to work in the test case. The focus still disappears into nowhere after date select. See video: http://screencast.com/t/hr8GYJpp.

Yip reports that the test still fails.

Yip, please make sure you have the latest code and any cache cleared. I'm pretty sure this time the fix is working well.

Use ice.ace.jq namespace instead of $ when referencing jQuery API.

[2:52:45 PM] Ken Fyten: Yip please retest http://jira.icesoft.org/browse/ICE-8558 with latest trunk.

• SVN Update: Completed: At revision: 32914
• ant clean
• TortoiseSVN -> Check for modifications -> Show all except Unmodified: make sure no files left over
• ant world
• close all FF tabs except Start Page, clear FF cache (Ctrl + Shift + Del -> Clear Everything), then exit FF
• run test app

• repeat all above 3 times

Still doesn't work. Same result as video shown above. Maybe get someone else to double-test?

I also just tested my original test case with the updated trunk and I could still see the issue as reported by Yip.

After doing a fresh checkout of ICEfaces I discovered that my last commit didn't make into the trunk. Committed successfully this time the change of $ function to ice.ace.jq in datetimeentry.js.

Please re-test.

Just tested the latest trunk with my test case and this looks to be resolved.