For PAYG, should we have a version of the API that worries about window scrolling and one that doesn't? If your app is designed not to scroll, I would think you wouldn't have to pay for this extra code?
-Alex On 6/1/18, 4:26 AM, "ha...@apache.org" <ha...@apache.org> wrote: This is an automated email from the ASF dual-hosted git repository. harbs pushed a commit to branch develop in repository https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgitbox.apache.org%2Frepos%2Fasf%2Froyale-asjs.git&data=02%7C01%7Caharui%40adobe.com%7C85aafe13907e4728298508d5c7b28caa%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C636634492071894168&sdata=upjYhaXnXu4zo0rSp731%2ByM89crQMlK%2BothTSTXQlqA%3D&reserved=0 commit e5328655b9991fe44b92c61d1b18aa7a4cc104b5 Author: Harbs <ha...@in-tools.com> AuthorDate: Tue May 29 12:25:55 2018 +0300 global point should ignore window scrolling --- .../royale/org/apache/royale/utils/PointUtils.as | 28 +++++++++++++--------- 1 file changed, 17 insertions(+), 11 deletions(-) diff --git a/frameworks/projects/Core/src/main/royale/org/apache/royale/utils/PointUtils.as b/frameworks/projects/Core/src/main/royale/org/apache/royale/utils/PointUtils.as index 264102c..f96e61c 100644 --- a/frameworks/projects/Core/src/main/royale/org/apache/royale/utils/PointUtils.as +++ b/frameworks/projects/Core/src/main/royale/org/apache/royale/utils/PointUtils.as @@ -71,18 +71,24 @@ package org.apache.royale.utils var x:Number = pt.x; var y:Number = pt.y; var element:HTMLElement = local.element as HTMLElement; + if ( element.getBoundingClientRect ) {// TODO take scrollbar widths into account + var rect:Object = element.getBoundingClientRect(); + x = x - rect.left - window.scrollX; + y = y - rect.top - window.scrollY; + } else { // for older browsers, but offsetParent is soon to be deprecated from chrome - do { - x -= element.offsetLeft; - y -= element.offsetTop; - if (local['parent'] !== undefined) { - local = local.parent; - element = local ? local.element as HTMLElement : null; - } else { - element = null; + do { + x -= element.offsetLeft; + y -= element.offsetTop; + if (local['parent'] !== undefined) { + local = local.parent; + element = local ? local.element as HTMLElement : null; + } else { + element = null; + } } + while (element); } - while (element); return new org.apache.royale.geom.Point(x, y); } @@ -119,8 +125,8 @@ package org.apache.royale.utils if ( element.getBoundingClientRect ) {// TODO take scrollbar widths into account var rect:Object = element.getBoundingClientRect(); - x = rect.left + x; - y = rect.top + y; + x = rect.left + x + window.scrollX; + y = rect.top + y + window.scrollY; } else { // for older browsers, but offsetParent is soon to be deprecated from from chrome do { x += element.offsetLeft; -- To stop receiving notification emails like this one, please contact ha...@apache.org.