[ http://issues.apache.org/jira/browse/ADFFACES-173?page=comments#action_12435910 ] Adam Winer commented on ADFFACES-173: -------------------------------------
Comments on the patch: - We should avoid checking agent types whereever possible in our rendering code; so only rendering _checkLoadNoPPR() on Blackberry is a problem; it should be rendered for all non-PPR browsers. - Can't we move the check for event being undefined into the JS? That is, "if (!event) event=window.event;" Don't put it in BodyRenderer. - Instead of checking "isBlackberry" before calling getElementById(), check if document.getElementById exists. > 'initialFocusId' attribute support for document component on PDA > ----------------------------------------------------------------- > > Key: ADFFACES-173 > URL: http://issues.apache.org/jira/browse/ADFFACES-173 > Project: MyFaces ADF-Faces > Issue Type: Improvement > Environment: PDA > Reporter: Piyush Hari > Priority: Minor > Attachments: patch_sept14_12_43PM.patch > > > 'initialFocusId' attribute on a 'document' component represents the id of the > component to which you want the focus to be when the full page renders. This > attribute is currently not supported on PDAs but works fine for desktop > applications. We plan to fix this. -- 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
