ICEfaces
  1. ICEfaces
  2. ICE-11444

ace:dataTable, livescrolling scrolls to a different row after loading new rows

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 4.3
    • Fix Version/s: EE-4.3.0.GA
    • Component/s: ACE-Components
    • Labels:
      None
    • Environment:
      Any

      Description

      When livescrolling is enabled on an ace:dataTable, the livescrolling functionality works correctly, regarding automatically loading subsequent or previous rows. However, after loading new rows, the scrollbar handle jumps to the top, displaying whatever rows currently appear at the top of the container, instead of displaying the same rows that were in view just before the livescrolling occurred.

      There's actually some code to do this already. However, something else is resetting the scrollbar to the top afterwards.

        Activity

        Arturo Zambrano created issue -
        Arturo Zambrano made changes -
        Field Original Value New Value
        Assignee Arturo Zambrano [ artzambrano ]
        Arturo Zambrano made changes -
        Fix Version/s EE-4.3.0.GA [ 13103 ]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #52840 Mon Dec 10 13:30:34 MST 2018 art.zambrano ICE-11444 modified the approach to adjust the scrollbar handle after the livescrolling occurs to endure that it is performed after all other resizing calculations
        Files Changed
        Commit graph MODIFY /icefaces4/trunk/icefaces/ace/component/src/org/icefaces/ace/component/datatable/DataTableRenderer.java
        Commit graph MODIFY /icefaces4/trunk/icefaces/ace/component/src/org/icefaces/ace/component/datatable/DataTable.java
        Commit graph MODIFY /icefaces4/trunk/icefaces/ace/component/resources/icefaces.ace/datatable/datatable.js
        Hide
        Arturo Zambrano added a comment -

        r52839, r52840: modified the approach to adjust the scrollbar handle after the livescrolling occurs to endure that it is performed after all other resizing calculations (committed to the 4.3 trunk and to the EE 4.3 tag)

        Show
        Arturo Zambrano added a comment - r52839, r52840: modified the approach to adjust the scrollbar handle after the livescrolling occurs to endure that it is performed after all other resizing calculations (committed to the 4.3 trunk and to the EE 4.3 tag)
        Arturo Zambrano made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #52853 Thu Dec 13 16:20:36 MST 2018 art.zambrano ICE-11444 added tabindex=0 to the scrollable body table to allow it to receive focus programmatically and control the scrollbar with the keyboard; adjusted the timeout delays for the functions that adjust the scrollbar to show the same rows that were showing before the livescrolling occurred
        Files Changed
        Commit graph MODIFY /icefaces4/trunk/icefaces/ace/component/src/org/icefaces/ace/component/datatable/DataTableRenderer.java
        Commit graph MODIFY /icefaces4/trunk/icefaces/ace/component/resources/icefaces.ace/datatable/datatable.js
        Hide
        Arturo Zambrano added a comment -

        r52853, r52854: added tabindex=0 to the scrollable body table to allow it to receive focus programmatically and control the scrollbar with the keyboard; adjusted the timeout delays for the functions that adjust the scrollbar to show the same rows that were showing before the livescrolling occurred (4.3 trunk and EE 4.3 tag)

        Show
        Arturo Zambrano added a comment - r52853, r52854: added tabindex=0 to the scrollable body table to allow it to receive focus programmatically and control the scrollbar with the keyboard; adjusted the timeout delays for the functions that adjust the scrollbar to show the same rows that were showing before the livescrolling occurred (4.3 trunk and EE 4.3 tag)
        Ken Fyten made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

          • Assignee:
            Arturo Zambrano
            Reporter:
            Arturo Zambrano
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: