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

              blocks

              Task - A task that needs to be done. ICE-8443 build ace:dataTable combined scrolling and editing test case

              • Major - Major loss of function.
              • Closed

                Activity

                Descending order - Click to sort in ascending order
                Ken Fyten made changes -
                Status Resolved [ 5 ] Closed [ 6 ]
                Ken Fyten made changes -
                Fix Version/s 3.2 [ 10338 ]
                Nils Lundquist made changes -
                Status Open [ 1 ] Resolved [ 5 ]
                Resolution Fixed [ 1 ]
                Hide
                Permalink
                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.
                Repository Revision Date User Message
                ICEsoft Public SVN Repository #30408 Wed Aug 08 11:43:43 MDT 2012 nils.lundquist ICE-8432 - Altered DataTable cell editor selectors to work with divs inserted for scrollable table sizing in the first row of the table.
                Files Changed
                Commit graph MODIFY /icefaces3/trunk/icefaces/ace/component/resources/icefaces.ace/datatable/datatable.js
                Nils Lundquist made changes -
                Link This issue blocks ICE-8443 [ ICE-8443 ]
                Arran Mccullough made changes -
                Assignee Nils Lundquist [ nils.lundquist ]
                Arran Mccullough made changes -
                Attachment Case11496Example.zip [ 14763 ]
                Attachment Case11496Example.war [ 14764 ]
                Hide
                Permalink
                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.
                Arran Mccullough made changes -
                Field Original Value New Value
                Salesforce Case [5007000000NHkQw]
                Arran Mccullough created issue -

                  People

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

                    Dates

                    • Created:
                      Updated:
                      Resolved: