If user specifies tabindex, should take user's tabindex? May not be 0.

Assume user won't set tabindex to < 0?

Need to adjust keyboard shortcuts and focus management code as well.

Revision: 24185

Modified : /icefaces2/trunk/icefaces/ace/component/resources/org.icefaces.component.datetimeentry/calendar.js
Modified : /icefaces2/trunk/icefaces/ace/component/src/org/icefaces/component/datetimeentry/DateTimeEntryMeta.java
Modified : /icefaces2/trunk/icefaces/ace/component/src/org/icefaces/component/datetimeentry/DateTimeEntryRenderer.java

An apparent regression from this change is that keyboard navigation (in default/ARIA mode) is no longer working at all with this component in the ace-showcase.

Regression not caused by this change, but by changes from ICE-6490. (this pointers didn't get all changed. As a result one of the this pointers got lost.)

Revision: 24209

Modified : /icefaces2/trunk/icefaces/ace/component/resources/org.icefaces.component.datetimeentry/calendar.js

From Ken:

One thing I noticed during testing is that if you have the dateTimeEntry in popup mode, when you use open the popup by clicking the button, it should move focus to the current date in the popup automatically. Right now there is no way to navigate into the popup calendar using the keyboard at all.

You do have a way to navigate by keyboard. The popup is a YUI dialog. When the dialog is rendered, focus is set by the dialog to one of its buttons. You just have to back tab into the calendar cells from the buttons. Need to find out if we can overcome this dialog focusing behavior.

The focus management is getting more and more involved. There is focusing by the framework, focusing by YUI and focusing by the component.

From Ken:

So long as the focus is set somewhere on the dialog that's good enough for 2.0.1.