ICEfaces
  1. ICEfaces
  2. ICE-8432

ace:rowEditor - Can't edit first row of a scrollable ace:dataTable

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 3.1
    • Fix Version/s: 3.2
    • Component/s: Documentation
    • Labels:
      None
    • Environment:
      All

      Description

      When using the ace:rowEditor/cellEditor in a scrollable ace:dataTable, clicking on the first rows "pencil" icon does not set the row in edit mode. All other rows work fine.

      Comment from customer on this issue:

      "This is caused by the fact that scrollable table inserts an extra div layer to specify width to the first row. This additional div is not counted for in datatable.js code. We have to override setupCellEditorEvents and doRowEditShowRequest to modify the selectors to make it work."

        Issue Links

          Activity

          Hide
          Arran Mccullough added a comment -

          Attached test case that reproduces the issue.

          Steps:

          • Load welcomeICEfaces.jsf
          • Try to edit the first row by clicking on the pencil icon. Nothing happens.
          Show
          Arran Mccullough added a comment - Attached test case that reproduces the issue. Steps: Load welcomeICEfaces.jsf Try to edit the first row by clicking on the pencil icon. Nothing happens.
          Hide
          Nils Lundquist added a comment -

          Revision #30408
          Committed by nils.lundquist
          A minute ago
          ICE-8432 - Altered DataTable cell editor selectors to work with divs inserted for scrollable table sizing in the first row of the table.

          Show
          Nils Lundquist added a comment - Revision #30408 Committed by nils.lundquist A minute ago ICE-8432 - Altered DataTable cell editor selectors to work with divs inserted for scrollable table sizing in the first row of the table.

            People

            • Assignee:
              Nils Lundquist
              Reporter:
              Arran Mccullough
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: