[ http://issues.apache.org/jira/browse/TAPESTRY-1102?page=comments#action_12438635 ] Greg Woolsey commented on TAPESTRY-1102: ----------------------------------------
Also, the ESC key should close the dialog. If I can figure out how to override the widget without exploding the whole jar file, I'll take a crack at a fix. I know our QA won't let us use 4.1.1 unless little things like this can be fixed, so I want to figure it out before I show them :) I've done both of these with the current dialog DHTML/JS we use in our app, but I want to remove that and use DoJo to reduce the client footprint if I can. I'll see if I can adapt any existing working key event code, but DoJo appears to have it's own framework used by the dojoAttachPoint template attributes, so that may be more a matter of finding their code to do the same thing and using it in AlertDialog. (Same with overriding the button look and feel - I need to make it fit our color scheme). > AlertDialog doesn't trap tabs or set focus to the button > -------------------------------------------------------- > > Key: TAPESTRY-1102 > URL: http://issues.apache.org/jira/browse/TAPESTRY-1102 > Project: Tapestry > Issue Type: Bug > Components: JavaScript > Affects Versions: 4.1.1 > Reporter: Greg Woolsey > > The AlertDialog widget used by the validation framework has a couple of > navigation issues: > 1) When it comes up, the "OK" button doesn't have the focus, but it appears > to the user to be the only active page element. > 2) Users can tab and shift-tab away from the dialog and enter data in form > fields behind the bakground iframe. This is non-intuitive, and almost always > undesired behavior. > It appears from the parent class, DoJo's Dialog.js that giving the button a > tabIndex, setting it as this.tabStart, and setting initial focus to it should > fix these. > Hard-core would be to trap all keystrokes and stop any key event that wasn't > space or enter, but that's probably more of a hammer than is needed. -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: http://issues.apache.org/jira/secure/Administrators.jspa - For more information on JIRA, see: http://www.atlassian.com/software/jira --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
