Oops, looks like I need to rebuild Core.  Ignore this.

________________________________
From: Yishay Weiss <yishayj...@hotmail.com>
Sent: Friday, April 17, 2020 8:05:06 PM
To: dev@royale.apache.org <dev@royale.apache.org>
Subject: RE: [royale-asjs] branch develop updated: jewel-scrollToIndex: remove 
isolated function

I’m getting

     [java] 
C:\dev\flexjs\royale-asjs\frameworks\projects\Jewel\src\main\royale\org\apache\royale\jewel\List.as(277):
 col: 20 Error: Access of possibly undefine
d property IScrollToIndexView.

When building Jewel.

I searched IScrollToIndexView and it doesn’t exist.

From: Carlos Rovira<mailto:carlosrov...@apache.org>
Sent: Friday, April 17, 2020 7:53 PM
To: Apache Royale Development<mailto:dev@royale.apache.org>
Subject: Re: [royale-asjs] branch develop updated: jewel-scrollToIndex: remove 
isolated function

Hi Yishay,

reverting the maven release plugin and rebuilding Jewel worked for me. I
don't see any problem for now

El vie., 17 abr. 2020 a las 17:43, Carlos Rovira (<carlosrov...@apache.org>)
escribió:

> Hi Yishay,
>
> the problem is I can't build due to develop in 0.9.8-SNAPSHOT
> I'll try locally to reset my repo to a commit where the versions are not
> bumped
>
> El vie., 17 abr. 2020 a las 14:44, Yishay Weiss (<yishayj...@hotmail.com>)
> escribió:
>
>> Is Jewel building for you? Even after that change it fails for me.
>>
>> ________________________________
>> From: Carlos Rovira <carlosrov...@apache.org>
>> Sent: Friday, April 17, 2020 3:34:21 PM
>> To: Apache Royale Development <dev@royale.apache.org>
>> Cc: comm...@royale.apache.org <comm...@royale.apache.org>
>> Subject: Re: [royale-asjs] branch develop updated: jewel-scrollToIndex:
>> remove isolated function
>>
>> Thanks
>> just commited :)
>>
>> El vie., 17 abr. 2020 a las 13:30, Yishay Weiss (<yishayj...@hotmail.com
>> >)
>> escribió:
>>
>> > Build is failing. Looks like you forgot to remove scrollToIndex from
>> > JewelClasses
>> >
>> > ________________________________
>> > From: carlosrov...@apache.org <carlosrov...@apache.org>
>> > Sent: Friday, April 17, 2020 3:56:44 AM
>> > To: comm...@royale.apache.org <comm...@royale.apache.org>
>> > 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 <carlosrov...@apache.org>
>> > 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;
>> > -               }
>> > -    }
>> > -}
>> >
>> >
>>
>> --
>> Carlos Rovira
>> http://about.me/carlosrovira
>>
>
>
> --
> Carlos Rovira
> http://about.me/carlosrovira
>
>

--
Carlos Rovira
http://about.me/carlosrovira

Reply via email to