[ 
https://issues.apache.org/jira/browse/TRINIDAD-1478?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12711064#action_12711064
 ] 

Matthias Weßendorf commented on TRINIDAD-1478:
----------------------------------------------

ok,
took a look at the patch.

when "focusNode " is null, why not doing this:

return "new _adfTreeSelector('" + selectParam + "'," + treeState + ");";

instead of this:

return "new _adfTreeSelector('" + selectParam + "'," + treeState + "," + 
focusNode + ");";

that way we would render out some smaller JS callbacks.

The generated function itself is fine with your change

> Tree initial focus state
> ------------------------
>
>                 Key: TRINIDAD-1478
>                 URL: https://issues.apache.org/jira/browse/TRINIDAD-1478
>             Project: MyFaces Trinidad
>          Issue Type: Improvement
>          Components: Components
>    Affects Versions: 1.0.10-core
>            Reporter: Tomas Havelka
>         Attachments: tree-focus-state.jpg, tree-focus.patch
>
>
> tr|tree component should be able to pass focus state (from getFocusRowKey 
> method) from server into client script when tree is rendered. In current tree 
> renderer implementation focus state is rendered correctly (means focused tree 
> style) but this node is not passed into client script thus tree can get into 
> inconsistent state (shown in image).

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to