ICEfaces
  1. ICEfaces
  2. ICE-866

Component Demo: TREE: Summarized list of problems

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.5
    • Fix Version/s: 1.6DR#1, 1.6
    • Component/s: Sample Apps
    • Labels:
      None
    • Environment:
      Operating System: Windows XP
      Platform: PC

      Description

      (1) User can't add nested nodes.
      (2) Tree nodes' presentation are required adjustments
      (3) Functionality Issue: User can't de-select a node once selected.
      (4) Add nodes till scroll bar appears. Now select a node and change the node
      type to either 'outputConnectionStatus' or 'outputProgress' and then move the
      scroll bar up and down.
      (5) In Mozilla, adding new nodes widens the tree combo box and eventually
      shifts the button box towards end of the screen.

        Activity

        Hide
        Saifullah Qazi added a comment -

        Some more:
        6) Add/Deleting a node erases the typed text in other nodes' edit boxes.
        7) The Sequence number generation is not that intelligent to keep the current
        nodes'quantity. (it increments without looking back how many are deleted)

        Show
        Saifullah Qazi added a comment - Some more: 6) Add/Deleting a node erases the typed text in other nodes' edit boxes. 7) The Sequence number generation is not that intelligent to keep the current nodes'quantity. (it increments without looking back how many are deleted)
        Hide
        Saifullah Qazi added a comment -

        8) Typing a word in text field of a node. Then clicking on 'Node' link erases
        the typed text.

        Show
        Saifullah Qazi added a comment - 8) Typing a word in text field of a node. Then clicking on 'Node' link erases the typed text.
        Hide
        Patrick Corless added a comment -

        I think the only item that should be looked at for the 1.6 release is item 6.

        Here's the scope for it.

        • Updated the tree node object so that a String label value can be set when
          someone enters text into a inputText object.
        • When a node is changed from outputText to inputText the "OutputText" label
          should be the value of the input field. And vise versa if the value of the
          outputText is changes.
        • If the node is changed to a command button then it's label should also be
          updated with the current label.

        To summaries, each node should store a notion of the a current label String.
        This string value should be displayed by the three different input types.

        Show
        Patrick Corless added a comment - I think the only item that should be looked at for the 1.6 release is item 6. Here's the scope for it. Updated the tree node object so that a String label value can be set when someone enters text into a inputText object. When a node is changed from outputText to inputText the "OutputText" label should be the value of the input field. And vise versa if the value of the outputText is changes. If the node is changed to a command button then it's label should also be updated with the current label. To summaries, each node should store a notion of the a current label String. This string value should be displayed by the three different input types.
        Hide
        Patrick Corless added a comment -

        re-asigning to Arran

        Show
        Patrick Corless added a comment - re-asigning to Arran
        Hide
        Arran McCullough added a comment -

        String value is now saved in each node, this value is kept when the component
        value is changed. When adding a component the default value is displayed,
        'OutputText', 'InputText', or 'Button'.

        Revision #13042

        Show
        Arran McCullough added a comment - String value is now saved in each node, this value is kept when the component value is changed. When adding a component the default value is displayed, 'OutputText', 'InputText', or 'Button'. Revision #13042
        Hide
        Patrick Corless added a comment -

        Code looks good but can fix up these two issues.

        1) TreeBean - can you add public javadoc to the changedValue(),
        componentTypeChanged(), updateNodeIcon() and addSelectedNode() methods.

        2) Also Yip reported the following build errors related to J2SE 1.5 syntax.
        Component showcase has to compile under J2SE 1.4.

        /home/builder/cruise-builds/checkout/icefaces-nightly/ce-bin-bundle/icefaces/samples/component-showcase/src/com/icesoft/icefaces/samples/showcase/components/tree/NodeUserObject.java:227:
        cannot resolve symbol
        symbol : method valueOf (int)
        location: class java.lang.Integer
        if(comp.equals(Integer.valueOf(1))){
        ^
        /home/builder/cruise-builds/checkout/icefaces-nightly/ce-bin-bundle/icefaces/samples/component-showcase/src/com/icesoft/icefaces/samples/showcase/components/tree/NodeUserObject.java:230:
        cannot resolve symbol
        symbol : method valueOf (int)
        location: class java.lang.Integer
        else if(comp.equals(Integer.valueOf(2))){
        ^
        2 errors

        Show
        Patrick Corless added a comment - Code looks good but can fix up these two issues. 1) TreeBean - can you add public javadoc to the changedValue(), componentTypeChanged(), updateNodeIcon() and addSelectedNode() methods. 2) Also Yip reported the following build errors related to J2SE 1.5 syntax. Component showcase has to compile under J2SE 1.4. /home/builder/cruise-builds/checkout/icefaces-nightly/ce-bin-bundle/icefaces/samples/component-showcase/src/com/icesoft/icefaces/samples/showcase/components/tree/NodeUserObject.java:227: cannot resolve symbol symbol : method valueOf (int) location: class java.lang.Integer if(comp.equals(Integer.valueOf(1))){ ^ /home/builder/cruise-builds/checkout/icefaces-nightly/ce-bin-bundle/icefaces/samples/component-showcase/src/com/icesoft/icefaces/samples/showcase/components/tree/NodeUserObject.java:230: cannot resolve symbol symbol : method valueOf (int) location: class java.lang.Integer else if(comp.equals(Integer.valueOf(2))){ ^ 2 errors
        Hide
        Arran McCullough added a comment -

        Fixed build error and added javadoc.

        Revision # 13046

        Show
        Arran McCullough added a comment - Fixed build error and added javadoc. Revision # 13046
        Hide
        Arran McCullough added a comment -

        Issue #6 Fixed on Revision #13046

        Show
        Arran McCullough added a comment - Issue #6 Fixed on Revision #13046

          People

          • Assignee:
            Unassigned
            Reporter:
            Saifullah Qazi
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: