ICEfaces
  1. ICEfaces
  2. ICE-8153

ace:list - dragging selected row item to new list results in selected row items in multiple lists concurrently

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 3.1.0.BETA1
    • Fix Version/s: 3.1.0.BETA2, 3.1
    • Component/s: ACE-Components
    • Labels:
      None
    • Environment:
      ace:list
    • Assignee Priority:
      P2
    • Affects:
      Documentation (User Guide, Ref. Guide, etc.), Compatibility/Configuration

      Description

      If you select multiple rows in list A, then drag a selected row to list B, you end of up with the remaining selected rows in list A as well as the dragged row selected in list B. Having selected rows in multiple lists concurrently is not a valid state.

      The solution here could be one of:

      1. When a row is dragged into another list, all previously selected rows from the originating list are deselected, and the dragged row is selected (in the new list). Also, if a row is dragged within the same list and dropped in a new location, all previously selected rows are deselected and that row is selected. Note that the selected event should fire before the move event in this case.

      2. When a row is dragged at all, it is selected and any other rows are deselected upon drag initiation. The problem with this approach is that you lose previously selected rows if you accidentally initiate a drag when you might have been meaning to click on a row. Also, if you initiate a drag and then drop the row in the same location in the list, you will lose previously selected rows. However, it does have the advantage of selecting the row when the mouse button is pressed over it, prior to the drag starting, which might be more consistent from a user-experience perspective.

      Either of these will also allow the select event to fire during a drag operation, which it currently does not as the row is not actually selected during the course of the drag operation.


        Activity

        Hide
        Nils Lundquist added a comment -

        I don't think it makes sense to deselect your selected items and force you to reselect them if you attempt to use drag drop to reorder them within a list.

        Show
        Nils Lundquist added a comment - I don't think it makes sense to deselect your selected items and force you to reselect them if you attempt to use drag drop to reorder them within a list.
        Hide
        Nils Lundquist added a comment -

        Revision #29374
        Committed by nils.lundquist
        A minute ago
        ICE-8153 - Drag & drop / double click migrations now deselect in all connected lists but the destination, and select the item being migrated.

        Show
        Nils Lundquist added a comment - Revision #29374 Committed by nils.lundquist A minute ago ICE-8153 - Drag & drop / double click migrations now deselect in all connected lists but the destination, and select the item being migrated.
        Hide
        Nils Lundquist added a comment -

        With the previous revision the drag behaviour as I believe it should be implemented is complete. By the grace of Jobs you may yet see the light and we will revert to this revision at a later date.

        I'm now finishing the other part of the functionality described above, where all drags, regardless of wether or not they migrate an item, deselect everything except themselves.

        Show
        Nils Lundquist added a comment - With the previous revision the drag behaviour as I believe it should be implemented is complete. By the grace of Jobs you may yet see the light and we will revert to this revision at a later date. I'm now finishing the other part of the functionality described above, where all drags, regardless of wether or not they migrate an item, deselect everything except themselves.
        Hide
        Nils Lundquist added a comment -

        Revision #29410
        Committed by nils.lundquist
        Moments ago
        ICE-8153 - Deselections on all drags and double click migration.

        ICE-8151 - Improvements to eventing. Only selected indexes have deselection added. Deselected lists during associated list deselection trigger a deselect event only if they have deselections to send.

        ICE-8234 - Selection property converted into SelectionMode.

        Show
        Nils Lundquist added a comment - Revision #29410 Committed by nils.lundquist Moments ago ICE-8153 - Deselections on all drags and double click migration. ICE-8151 - Improvements to eventing. Only selected indexes have deselection added. Deselected lists during associated list deselection trigger a deselect event only if they have deselections to send. ICE-8234 - Selection property converted into SelectionMode.

          People

          • Assignee:
            Nils Lundquist
            Reporter:
            Ken Fyten
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: