You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@royale.apache.org by ah...@apache.org on 2020/02/21 23:57:50 UTC

[royale-asjs] branch develop updated: workaround for chrome scrolling issue sort of described here https://support.google.com/chrome/thread/15069113?hl=en. Should fix #724

This is an automated email from the ASF dual-hosted git repository.

aharui pushed a commit to branch develop
in repository https://gitbox.apache.org/repos/asf/royale-asjs.git


The following commit(s) were added to refs/heads/develop by this push:
     new 2719699  workaround for chrome scrolling issue sort of described here https://support.google.com/chrome/thread/15069113?hl=en.  Should fix #724
2719699 is described below

commit 271969943a51cd49de8e396aef26d123ea04e53a
Author: Alex Harui <ah...@apache.org>
AuthorDate: Fri Feb 21 15:57:31 2020 -0800

    workaround for chrome scrolling issue sort of described here https://support.google.com/chrome/thread/15069113?hl=en.  Should fix #724
---
 .../main/royale/mx/controls/beads/layouts/AdvancedDataGridLayout.as    | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/frameworks/projects/MXRoyale/src/main/royale/mx/controls/beads/layouts/AdvancedDataGridLayout.as b/frameworks/projects/MXRoyale/src/main/royale/mx/controls/beads/layouts/AdvancedDataGridLayout.as
index 231830b..f2076b1 100644
--- a/frameworks/projects/MXRoyale/src/main/royale/mx/controls/beads/layouts/AdvancedDataGridLayout.as
+++ b/frameworks/projects/MXRoyale/src/main/royale/mx/controls/beads/layouts/AdvancedDataGridLayout.as
@@ -149,6 +149,7 @@ package mx.controls.beads.layouts
                 COMPILE::JS
                 {
                 firstVisibleIndex = Math.floor(listArea.element.scrollTop / actualRowHeight);
+				var scrollTop:Number = listArea.element.scrollTop;
                 var topSpacerHeight:Number = Math.floor(listArea.element.scrollTop / actualRowHeight)
                     * actualRowHeight;
                 }
@@ -197,6 +198,8 @@ package mx.controls.beads.layouts
                     {
                         columnList.element.style.position = "absolute";
                         columnList.element.style.top = (topSpacerHeight + 1).toString() + 'px';
+						// chrome has bug where moving things resets scrollTop
+						listArea.element.scrollTop = scrollTop;
                         columnList.dispatchEvent(new Event("layoutNeeded"));
                     }
                 }