[ http://issues.apache.org/jira/browse/TAPESTRY-1102?page=comments#action_12438764 ] Jesse Kuhnert commented on TAPESTRY-1102: -----------------------------------------
Ahh I see now... Yeah, I may need to remove that particular button altogether. It's not actually a native browser button so it doesn't even have a "focus" function to call. I don't have time to do a thorough job of implementing this change right this second but will re-open this ticket as a marker so I know to come back and do it.. P.S. If you ~do~ decide to create your own Dialog please feel free to send in a patch (if that's feasible, you may want to do more things specific to your app which is fine..) . > 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 > Assigned To: Jesse Kuhnert > Fix For: 4.1.1 > > > 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]
