ICEfaces
  1. ICEfaces
  2. ICE-10489

ace:tree - Node alignment issue

    Details

      Description

      When displaying an ace:tree with a mixture of nodes with children and nodes with no children. The leaf nodes are rendered misaligned with the nodes with children nodes.

      This is caused by the leaf nodes not rendering the expand/contract icon (+/-).
      1. Case13296Example.zip
        23 kB
        Arran Mccullough
      2. Case13296ExampleWAR.zip
        9.32 MB
        Arran Mccullough
      1. tree-node-alignment.jpg
        61 kB
      2. screenshot-ice10489-gap.png
        35 kB
      3. screenshot-ice10489-no-gap.png
        35 kB

        Activity

        Hide
        Arran Mccullough added a comment -

        Attached test case and image that shows this issue.

        Show
        Arran Mccullough added a comment - Attached test case and image that shows this issue.
        Hide
        Arturo Zambrano added a comment -

        Committed fix to 4.0 trunk at revision 44064 and to 3.3 EE maintenance branch at revision 44065.

        Added margin to root leaf nodes so they align with root nodes that have children (which display a contracted/expanded icon). This margin is only applied to root nodes. When it was applied to child nodes, it actually didn't look well, since it formed a gap between the tree "branch" and the label and it could also lead to confusion as appearing to be under a sibling that has the contracted icon.

        Show
        Arturo Zambrano added a comment - Committed fix to 4.0 trunk at revision 44064 and to 3.3 EE maintenance branch at revision 44065. Added margin to root leaf nodes so they align with root nodes that have children (which display a contracted/expanded icon). This margin is only applied to root nodes. When it was applied to child nodes, it actually didn't look well, since it formed a gap between the tree "branch" and the label and it could also lead to confusion as appearing to be under a sibling that has the contracted icon.
        Hide
        Carmen Cristurean added a comment -

        Verified in IE11, Chrome40, FF34, with IF4 trunk r44097 and EE-3.3.0-maintenance branch r44065.

        Show
        Carmen Cristurean added a comment - Verified in IE11, Chrome40, FF34, with IF4 trunk r44097 and EE-3.3.0-maintenance branch r44065.
        Hide
        Arturo Zambrano added a comment -

        Added left margin to all leaf nodes in the tree (not only roots) at revisions 44111 and 44112.

        Testing notes: please verify again, this time all child nodes will also have a margin.

        Show
        Arturo Zambrano added a comment - Added left margin to all leaf nodes in the tree (not only roots) at revisions 44111 and 44112. Testing notes: please verify again, this time all child nodes will also have a margin.

          People

          • Assignee:
            Arturo Zambrano
            Reporter:
            Arran Mccullough
          • Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: