ICEfaces
  1. ICEfaces
  2. ICE-8764

ace:draggable - Add a drag start event listener

          Details

          • Type: Improvement Improvement
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: EE-3.0.0.GA_P01, 3.2
          • Fix Version/s: EE-3.2.0.BETA, EE-3.2.0.GA, 3.3
          • Component/s: ACE-Components
          • Labels:
            None
          • Environment:
            All
          • Affects:
            Documentation (User Guide, Ref. Guide, etc.), Sample App./Tutorial

            Description


            Feature Request: Add a drag start event so a listener could be fired when the ace:draggable object is first dragged.

              Activity

              Ascending order - Click to sort in descending order
              Migration created issue -
              Hide
              Permalink
              Arran Mccullough added a comment - - edited

              Comment from Art on this functionality:
              "jQuery UI does provide a hook for the drag start event. So, we can definitely add this feature.
              The only thing to consider is that since this event fires when the user starts dragging an element, and it's very likely that the user will still be dragging the element when the response comes back, there might be some minor usability issues if the response modifies the component being dragged or the draggable object itself, like losing the grip of the element. We fixed a similar situation in slider, so we might be able to fix this situation for draggables as well, if we make some necessary changes in the rendering. However, in some cases it may depend on how the component being dragged is rendered. "
              (Restricted to icesoft-internal-developers group)

              Show
              Arran Mccullough added a comment - - edited Comment from Art on this functionality: "jQuery UI does provide a hook for the drag start event. So, we can definitely add this feature. The only thing to consider is that since this event fires when the user starts dragging an element, and it's very likely that the user will still be dragging the element when the response comes back, there might be some minor usability issues if the response modifies the component being dragged or the draggable object itself, like losing the grip of the element. We fixed a similar situation in slider, so we might be able to fix this situation for draggables as well, if we make some necessary changes in the rendering. However, in some cases it may depend on how the component being dragged is rendered. " (Restricted to icesoft-internal-developers group)
              Hide
              Permalink
              Arturo Zambrano added a comment - - edited

              I estimate this can take half a day (4 hours) to implement, test, and also to analyze different situations that can arise with this feature like the one described in the first comment.
              (Restricted to icesoft-internal-developers group)

              Show
              Arturo Zambrano added a comment - - edited I estimate this can take half a day (4 hours) to implement, test, and also to analyze different situations that can arise with this feature like the one described in the first comment. (Restricted to icesoft-internal-developers group)
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #32264 Tue Nov 20 15:27:57 MST 2012 art.zambrano ICE-8764 ace:draggable, added 'dragStartListener' attribute and 'start' ajax event
              Files Changed
              Commit graph MODIFY /icefaces3/trunk/icefaces/ace/component/src/org/icefaces/ace/component/dnd/DraggableMeta.java
              Commit graph MODIFY /icefaces3/trunk/icefaces/ace/component/src/org/icefaces/ace/component/dnd/Draggable.java
              Commit graph MODIFY /icefaces3/trunk/icefaces/ace/component/resources/icefaces.ace/dnd/dragdrop.js
              Commit graph MODIFY /icefaces3/trunk/icefaces/ace/component/src/org/icefaces/ace/component/dnd/DraggableRenderer.java
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #32526 Thu Nov 29 16:08:43 MST 2012 art.zambrano ICE-8764 ace:draggable, added 'datasource' attribute to work with 'dragStartListener'
              Files Changed
              Commit graph MODIFY /icefaces3/trunk/icefaces/ace/component/src/org/icefaces/ace/component/dnd/DraggableMeta.java
              Commit graph MODIFY /icefaces3/trunk/icefaces/ace/component/resources/icefaces.ace/dnd/dragdrop.js
              Commit graph MODIFY /icefaces3/trunk/icefaces/ace/component/src/org/icefaces/ace/component/dnd/DraggableRenderer.java
              Migration made changes -
              Field Original Value New Value
              Reporter Migration [ remote ] Arran Mccullough [ arran.mccullough ]
              Migration made changes -
              Assignee Arturo Zambrano [ artzambrano ]
              Migration made changes -
              Fix Version/s 3.3 [ 10370 ]
              Migration made changes -
              Fix Version/s EE-3.2.0.GA [ 10332 ]
              Fix Version/s 3.3 [ 10370 ]
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #32549 Fri Nov 30 12:25:48 MST 2012 art.zambrano ICE-8764 ace:draggable, expanded tld documentation for 'dragStartListener'
              Files Changed
              Commit graph MODIFY /icefaces3/trunk/icefaces/ace/component/src/org/icefaces/ace/component/dnd/DraggableMeta.java
              Migration made changes -
              Fix Version/s 3.3 [ 10370 ]
              Fix Version/s EE-3.2.0.GA [ 10332 ]
              Migration made changes -
              Fix Version/s EE-3.2.0.GA [ 10332 ]
              Fix Version/s 3.3 [ 10370 ]
              Hide
              Permalink
              Arturo Zambrano added a comment - - edited

              Committed feature to the trunk at revision at 32526.

              Added 'dragStartListener' attribute and 'start' ajax event, as well as a 'datasource' attribute to fetch data when working with draggables in UIData components.

              Show
              Arturo Zambrano added a comment - - edited Committed feature to the trunk at revision at 32526. Added 'dragStartListener' attribute and 'start' ajax event, as well as a 'datasource' attribute to fetch data when working with draggables in UIData components.
              Arturo Zambrano made changes -
              Status Open [ 1 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Ken Fyten made changes -
              Fix Version/s 3.3 [ 10370 ]
              Ken Fyten made changes -
              Affects Documentation (User Guide, Ref. Guide, etc.),Sample App./Tutorial [ 10003, 10001 ]
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #32852 Tue Dec 18 10:42:21 MST 2012 art.zambrano ICE-8764 change to avoid relying on the node passed to the event handler for obtaining the id of the dragged component, which was causing problems when using helper="clone"
              Files Changed
              Commit graph MODIFY /icefaces3/trunk/icefaces/ace/component/resources/icefaces.ace/dnd/dragdrop.js
              Hide
              Permalink
              Arturo Zambrano added a comment - - edited

              There was a problem when using helper="clone". Since a clone node is created from the original node, the id attribute is not included (otherwise it would be duplicate), so when retrieving the id from the clone node, during the dragStart event, it was undefined and it wasn't possible to determine which node had been dragged.

              Committed fix for this problem at revision 32852 to avoid relying on the id of the clone node, but on the target id specified initially.

              Show
              Arturo Zambrano added a comment - - edited There was a problem when using helper="clone". Since a clone node is created from the original node, the id attribute is not included (otherwise it would be duplicate), so when retrieving the id from the clone node, during the dragStart event, it was undefined and it wasn't possible to determine which node had been dragged. Committed fix for this problem at revision 32852 to avoid relying on the id of the clone node, but on the target id specified initially.
              Ken Fyten made changes -
              Fix Version/s EE-3.2.0.BETA [ 10573 ]
              Ken Fyten made changes -
              Status Resolved [ 5 ] Closed [ 6 ]

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved: