ICEfaces
  1. ICEfaces
  2. ICE-3983

ice:tree renders navigation line when ice:treeNode render=false

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 1.7.2
    • Fix Version/s: 1.8.1
    • Component/s: ICE-Components
    • Labels:
      None
    • Environment:
      N/A

      Description

      When rendered by ice:treeNode is false (if tree.isNodeVisible(node) == false) navigation lines to non-rendered nodes are visible.


      <ice:tree id="tree"
                        value="#{tree.model}"
                        var="node"
                        hideRootNode="true"
                        hideNavigation="false"
                        imageDir="#{styleBean.imageDirectory}" >

                  <ice:treeNode rendered="#{tree.isNodeVisible(node)}">
                      <f:facet name="icon">
                          <ice:panelGroup styleClass="treeNode">
                              <ice:graphicImage value="/xmlhttp/css/#{styleBean.currentStyle}/css-images/#{node.userObject.icon}" />
                          </ice:panelGroup>
                      </f:facet>
                      <f:facet name="content">
                          <ice:commandLink action="#{tree.setSelectedNode(node)}"
                                           styleClass="selectedNode#{node eq tree.selectedNode} treeText" >
                              <ice:outputText id="TreeNode"
                                              value="#{node.userObject.text}" />
                          </ice:commandLink>
                      </f:facet>

                  </ice:treeNode>
              </ice:tree>
      1. ICE-3983.patch
        1 kB
        Ken Fyten
      1. ScreenHunter_04.jpg
        155 kB
      2. tree.jpg
        5 kB

        Activity

        Petra Filipova created issue -
        Petra Filipova made changes -
        Field Original Value New Value
        Attachment tree.jpg [ 11456 ]
        Hide
        Ken Fyten added a comment -

        Please review the contribution and commit if appropriate, etc.

        Show
        Ken Fyten added a comment - Please review the contribution and commit if appropriate, etc.
        Ken Fyten made changes -
        Salesforce Case []
        Fix Version/s 1.8.1 [ 10170 ]
        Assignee Priority P3
        Assignee Yip Ng [ yip.ng ]
        Priority Major [ 3 ] Minor [ 4 ]
        Hide
        Ken Fyten added a comment -

        Contributed patch from Stefen Niederhausen @ Mimacom.

        Show
        Ken Fyten added a comment - Contributed patch from Stefen Niederhausen @ Mimacom.
        Ken Fyten made changes -
        Attachment ICE-3983.patch [ 11720 ]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #18850 Thu May 07 11:47:31 MDT 2009 yip.ng ICE-3983
        Contributed patch from Stefen Niederhausen @ Mimacom.
        Files Changed
        Commit graph MODIFY /icefaces/trunk/icefaces/component/src/com/icesoft/faces/component/tree/TreeRenderer.java
        yip.ng made changes -
        Attachment ScreenHunter_04.jpg [ 11723 ]
        Hide
        yip.ng added a comment -

        Patch applied and tested. See attached ScreenHunter_04.jpg.

        Show
        yip.ng added a comment - Patch applied and tested. See attached ScreenHunter_04.jpg.
        yip.ng made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Ken Fyten made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Assignee Priority P3
        Hide
        ?ukasz Pogorzelski added a comment -

        As you can see on attached picture the problem haven't been resolved yet. There is an unfinished navigation line under node "Garcia, Benjamin". I've tested this on version 1.8.2. So you can think about to reopen this issue.

        Show
        ?ukasz Pogorzelski added a comment - As you can see on attached picture the problem haven't been resolved yet. There is an unfinished navigation line under node "Garcia, Benjamin". I've tested this on version 1.8.2. So you can think about to reopen this issue.

          People

          • Assignee:
            yip.ng
            Reporter:
            Petra Filipova
          • Votes:
            2 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: