ICEfaces
  1. ICEfaces
  2. ICE-7269

Enhance ace:progressBar: Add listener for current progress

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.1-Beta
    • Fix Version/s: 2.1-Beta2, 3.0
    • Component/s: ACE-Components
    • Labels:
      None
    • Environment:
      All
    • Assignee Priority:
      P2

      Description

      This component could use a listener so that the current progress can be displayed (like % of completion for example).

        Activity

        Cruz Miraback created issue -
        Cruz Miraback made changes -
        Field Original Value New Value
        Assignee Ken Fyten [ ken.fyten ]
        Ken Fyten made changes -
        Salesforce Case []
        Fix Version/s 2.1 [ 10241 ]
        Assignee Priority P2
        Affects Version/s 2.1-Beta [ 10291 ]
        Assignee Ken Fyten [ ken.fyten ] yip.ng [ yip.ng ]
        Hide
        yip.ng added a comment - - edited

        Done. See video at http://screencast.com/t/bJbGoswZrc.

        Notice that in the comp suite demo, the progress bar starts with a value of 10 instead of 0. That's because the value is incremented in the getter. The getter is not just called by encode(), it is also called by decode() and maybe the framework (e.g. tag handler?). So the value shouldn't be changed in the getter. Ideally in this case it should be changed by a separate thread.

        Also in this use case scenario, don't instinctively try to set the value on a change event, or you will end up with an infinite loop forever triggering change events.

        Revision: 25917


        Modified : /icefaces-ee/scratchpads/grimlock/icefaces/ace/component/resources/icefaces.ace/jquery/ui/jquery-ui.js
        Modified : /icefaces-ee/scratchpads/grimlock/icefaces/ace/component/resources/icefaces.ace/progressbar/progressbar.js
        Modified : /icefaces-ee/scratchpads/grimlock/icefaces/ace/component/src/org/icefaces/ace/component/progressbar/ProgressBarMeta.java
        Modified : /icefaces-ee/scratchpads/grimlock/icefaces/ace/component/src/org/icefaces/ace/component/progressbar/ProgressBarRenderer.java
        Added : /icefaces-ee/scratchpads/grimlock/icefaces/ace/component/src/org/icefaces/ace/event/ProgressBarChangeEvent.java

        Revision: 29025


        Modified : /applications/comp-suite/trunk/comp-suite/src/main/java/org/icefaces/samples/showcase/example/ace/progressbar/ProgressBarBean.java
        Modified : /applications/comp-suite/trunk/comp-suite/src/main/resources/org/icefaces/samples/showcase/view/resources/messages.properties
        Modified : /applications/comp-suite/trunk/comp-suite/src/main/webapp/resources/examples/ace/progressbar/progressBar.xhtml

        Show
        yip.ng added a comment - - edited Done. See video at http://screencast.com/t/bJbGoswZrc . Notice that in the comp suite demo, the progress bar starts with a value of 10 instead of 0. That's because the value is incremented in the getter. The getter is not just called by encode(), it is also called by decode() and maybe the framework (e.g. tag handler?). So the value shouldn't be changed in the getter. Ideally in this case it should be changed by a separate thread. Also in this use case scenario, don't instinctively try to set the value on a change event, or you will end up with an infinite loop forever triggering change events. Revision: 25917 Modified : /icefaces-ee/scratchpads/grimlock/icefaces/ace/component/resources/icefaces.ace/jquery/ui/jquery-ui.js Modified : /icefaces-ee/scratchpads/grimlock/icefaces/ace/component/resources/icefaces.ace/progressbar/progressbar.js Modified : /icefaces-ee/scratchpads/grimlock/icefaces/ace/component/src/org/icefaces/ace/component/progressbar/ProgressBarMeta.java Modified : /icefaces-ee/scratchpads/grimlock/icefaces/ace/component/src/org/icefaces/ace/component/progressbar/ProgressBarRenderer.java Added : /icefaces-ee/scratchpads/grimlock/icefaces/ace/component/src/org/icefaces/ace/event/ProgressBarChangeEvent.java Revision: 29025 Modified : /applications/comp-suite/trunk/comp-suite/src/main/java/org/icefaces/samples/showcase/example/ace/progressbar/ProgressBarBean.java Modified : /applications/comp-suite/trunk/comp-suite/src/main/resources/org/icefaces/samples/showcase/view/resources/messages.properties Modified : /applications/comp-suite/trunk/comp-suite/src/main/webapp/resources/examples/ace/progressbar/progressBar.xhtml
        yip.ng made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Ken Fyten made changes -
        Fix Version/s 2.1-Beta2 [ 10294 ]
        Ken Fyten made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

          • Assignee:
            yip.ng
            Reporter:
            Cruz Miraback
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: