ICEfaces
  1. ICEfaces
  2. ICE-9258

ace:dataTable - Add new "rowClick" and "rowDblClick" ace:ajax events

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 4.0.BETA, EE-3.3.0.GA_P02, 4.0
    • Component/s: ACE-Components
    • Labels:
      None
    • Environment:
      ICEfaces 3.x ace:dataTable
    • Assignee Priority:
      P2
    • Salesforce Case Reference:
    • Affects:
      Documentation (User Guide, Ref. Guide, etc.)

      Description

      Users would like to ability to configure the ace:dataTable to say, use single-click for row selection, and also add a double-click listener to the row that would perform a different action, such as popup an edit dialog, etc.

      In order to facilitate this, we should add the following new client events / ace:ajax events to ace:dataTable:

      * 'rowClick' - fires when the primary mouse button is clicked once over a table row.
      * "rowDblClick" - fires when the primary mouse button is double-clicked over a table row.

      In to support the "rowDblClick" detection, a technique similar to that used by the ace:list may be required.

      Note that it is also foreseeable that the application may want to first select the double-clicked row, then perform some action on it. Presumably, this could be implemented by the application "rowDblClick" event handler itself, and not in the ace:dataTable component (unless row selection is already configured for double-click).

        Activity

        Hide
        Nils Lundquist added a comment -

        Revision #35930
        Committed by nils.lundquist
        2 minutes ago
        ICE-9258 - Added DataTable cellClick & cellDoubleClick events. Refactored DataTable row/cell click event binding.

        Show
        Nils Lundquist added a comment - Revision #35930 Committed by nils.lundquist 2 minutes ago ICE-9258 - Added DataTable cellClick & cellDoubleClick events. Refactored DataTable row/cell click event binding.
        Hide
        Nils Lundquist added a comment - - edited

        Reopening issue to add row/cell context info to ace:ajax events.

        Show
        Nils Lundquist added a comment - - edited Reopening issue to add row/cell context info to ace:ajax events.
        Hide
        Nils Lundquist added a comment -

        Revision #36191
        Committed by nils.lundquist
        A minute ago
        ICE-9258 - Added DataTableCellClickEvent AjaxBehaviourEvent subclass.

        Show
        Nils Lundquist added a comment - Revision #36191 Committed by nils.lundquist A minute ago ICE-9258 - Added DataTableCellClickEvent AjaxBehaviourEvent subclass.
        Hide
        Nils Lundquist added a comment -

        The functionality added as part of this JIRA will have to have extensive tests added to confirm:

        • the right column / row are always part of the DataTableCellClickEvent.
        • no double submits occur when using single & double click
        • standard ace:ajax testing
        Show
        Nils Lundquist added a comment - The functionality added as part of this JIRA will have to have extensive tests added to confirm: the right column / row are always part of the DataTableCellClickEvent. no double submits occur when using single & double click standard ace:ajax testing
        Hide
        Ken Fyten added a comment -
        • Optimize so delay is only used if double click event is listened for.
        • Add new showcase demo to show single click row selection mode with dbl click listener to show popup.
        Show
        Ken Fyten added a comment - Optimize so delay is only used if double click event is listened for. Add new showcase demo to show single click row selection mode with dbl click listener to show popup.
        Hide
        Nils Lundquist added a comment -

        Revision #36293
        Committed by nils.lundquist
        A minute ago
        ICE-9258 - Added showcase example of DataTable ajax row click behaviours.

        Show
        Nils Lundquist added a comment - Revision #36293 Committed by nils.lundquist A minute ago ICE-9258 - Added showcase example of DataTable ajax row click behaviours.
        Hide
        Arturo Zambrano added a comment -

        Finished backporting at revision 38772.

        Show
        Arturo Zambrano added a comment - Finished backporting at revision 38772.

          People

          • Assignee:
            Arturo Zambrano
            Reporter:
            Ken Fyten
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: