Build is failing. Looks like you forgot to remove scrollToIndex from 
JewelClasses

________________________________
From: [email protected] <[email protected]>
Sent: Friday, April 17, 2020 3:56:44 AM
To: [email protected] <[email protected]>
Subject: [royale-asjs] branch develop updated: jewel-scrollToIndex: remove 
isolated function

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

carlosrovira 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 8f66e33  jewel-scrollToIndex: remove isolated function
8f66e33 is described below

commit 8f66e334aebb23c6bf802232de6f925a9e4a2a13
Author: Carlos Rovira <[email protected]>
AuthorDate: Fri Apr 17 02:56:40 2020 +0200

    jewel-scrollToIndex: remove isolated function
---
 .../jewel/beads/controls/list/scrollToIndex.as     | 97 ----------------------
 1 file changed, 97 deletions(-)

diff --git 
a/frameworks/projects/Jewel/src/main/royale/org/apache/royale/jewel/beads/controls/list/scrollToIndex.as
 
b/frameworks/projects/Jewel/src/main/royale/org/apache/royale/jewel/beads/controls/list/scrollToIndex.as
deleted file mode 100644
index fd4f278..0000000
--- 
a/frameworks/projects/Jewel/src/main/royale/org/apache/royale/jewel/beads/controls/list/scrollToIndex.as
+++ /dev/null
@@ -1,97 +0,0 @@
-////////////////////////////////////////////////////////////////////////////////
-//
-//  Licensed to the Apache Software Foundation (ASF) under one or more
-//  contributor license agreements.  See the NOTICE file distributed with
-//  this work for additional information regarding copyright ownership.
-//  The ASF licenses this file to You under the Apache License, Version 2.0
-//  (the "License"); you may not use this file except in compliance with
-//  the License.  You may obtain a copy of the License at
-//
-//      http://www.apache.org/licenses/LICENSE-2.0
-//
-//  Unless required by applicable law or agreed to in writing, software
-//  distributed under the License is distributed on an "AS IS" BASIS,
-//  WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-//  See the License for the specific language governing permissions and
-//  limitations under the License.
-//
-////////////////////////////////////////////////////////////////////////////////
-package org.apache.royale.jewel.beads.controls.list
-{
-       import org.apache.royale.core.IItemRenderer;
-       import org.apache.royale.core.IItemRendererOwnerView;
-       import org.apache.royale.core.IStrand;
-       import org.apache.royale.html.beads.IListView;
-       import org.apache.royale.jewel.List;
-       import org.apache.royale.jewel.beads.models.ListPresentationModel;
-
-       /**
-     *  Ensures that the data provider item at the given index is visible.
-     *
-     *  If the item is visible, the <code>verticalScrollPosition</code>
-     *  property is left unchanged even if the item is not the first visible
-     *  item. If the item is not currently visible, the
-     *  <code>verticalScrollPosition</code>
-     *  property is changed make the item the first visible item, unless there
-     *  aren't enough rows to do so because the
-     *  <code>verticalScrollPosition</code> value is limited by the
-     *  <code>maxVerticalScrollPosition</code> property.
-     *
-     *  @param index The index of the item in the data provider.
-     *
-     *  @return <code>true</code> if <code>verticalScrollPosition</code> 
changed.
-     *
-     *  @langversion 3.0
-     *  @playerversion Flash 9
-     *  @playerversion AIR 1.1
-     *  @productversion Royale 0.9.7
-     */
-    public function scrollToIndex(list:IStrand, index:int):Boolean
-    {
-        var _list:List = list as List;
-
-        COMPILE::SWF
-        {
-            // to implement
-
-            return false;
-        }
-
-               COMPILE::JS
-               {
-        var scrollArea:HTMLElement = _list.element;
-        var oldScroll:Number = scrollArea.scrollTop;
-
-        var totalHeight:Number = 0;
-
-        if(_list.variableRowHeight)
-        {
-            var listView:IListView = list.getBeadByType(IListView) as 
IListView;
-            var dataGroup:IItemRendererOwnerView = listView.dataGroup;
-
-            //each item render can have its own height
-            var n:int = _list.dataProvider.length;
-            var irHeights:Array = [];
-            for (var i:int = 0; i <= index; i++)
-            {
-                var ir:IItemRenderer = dataGroup.getItemRendererForIndex(i) as 
IItemRenderer;
-                totalHeight += ir.element.clientHeight;
-                irHeights.push(totalHeight + ir.element.clientHeight - 
scrollArea.clientHeight);
-            }
-
-            scrollArea.scrollTop = Math.min(irHeights[index], totalHeight);
-
-        } else
-        {
-            var rowHeight:Number;
-            // all items renderers with same height
-            rowHeight = isNaN(_list.rowHeight) ? 
ListPresentationModel.DEFAULT_ROW_HEIGHT : _list.rowHeight;
-            totalHeight = _list.dataProvider.length * rowHeight - 
scrollArea.clientHeight;
-
-            scrollArea.scrollTop = Math.min(index * rowHeight, totalHeight);
-        }
-
-        return oldScroll != scrollArea.scrollTop;
-               }
-    }
-}

Reply via email to