[jira] Commented: (ADFFACES-173) 'initialFocusId' attribute support for document component on PDA

["Adam Winer (JIRA)" <ad...@incubator.apache.org>]

    [ 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