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

                Ascending order - Click to sort in descending order
                Arran Mccullough created issue -
                Arran Mccullough made changes -
                Field Original Value New Value
                Salesforce Case [5007000000NHkQw]
                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 -
                Attachment Case11496Example.zip [ 14763 ]
                Attachment Case11496Example.war [ 14764 ]
                Arran Mccullough made changes -
                Assignee Nils Lundquist [ nils.lundquist ]
                Nils Lundquist made changes -
                Link This issue blocks ICE-8443 [ ICE-8443 ]
                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
                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.
                Nils Lundquist made changes -
                Status Open [ 1 ] Resolved [ 5 ]
                Resolution Fixed [ 1 ]
                Ken Fyten made changes -
                Fix Version/s 3.2 [ 10338 ]
                Ken Fyten made changes -
                Status Resolved [ 5 ] Closed [ 6 ]

                  People

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

                    Dates

                    • Created:
                      Updated:
                      Resolved: