Ok,  I missed that, but there is sort of a template we should follow.  See the 
other MIT Licenses in LICENSE.

Which SVG files did you use in the LoadIndicators?  IMO, a comment in the 
source file indicating that the SVG content is third-party is worth doing.

Thanks,
-Alex

On 6/19/19, 12:27 AM, "Yishay Weiss" <[email protected]> wrote:

    I changed that in
    
    
    
    a45f3638697492642ee93c60e74f8995dfcf79a9
    
    
    
    Thanks.
    
    
    
    
    
    ________________________________
    From: Alex Harui <[email protected]>
    Sent: Wednesday, June 19, 2019 9:20:47 AM
    To: [email protected]; [email protected]
    Subject: Re: [royale-asjs] branch develop updated: Adding a resizable load 
inicator.
    
    Yishay,
    
    Are you still using 3rd party load indicators?  If so, I think the MIT 
License attribution is actually supposed to go in LICENSE not NOTICE.  See
    
https://nam04.safelinks.protection.outlook.com/?url=http%3A%2F%2Fwww.apache.org%2Fdev%2Flicensing-howto.html%23permissive-deps&amp;data=02%7C01%7Caharui%40adobe.com%7Ceef5f6c17aa24158479308d6f487a3bf%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C636965260739304265&amp;sdata=i0JKUFcfuLk9ZxXTdnU1ALH5E5QevK9IbiKqKw0c0z8%3D&amp;reserved=0
    
    Thanks,
    -Alex
    
    On 6/18/19, 10:18 PM, "[email protected]" <[email protected]> wrote:
    
        This is an automated email from the ASF dual-hosted git repository.
    
        yishayw pushed a commit to branch develop
        in repository 
https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgitbox.apache.org%2Frepos%2Fasf%2Froyale-asjs.git&amp;data=02%7C01%7Caharui%40adobe.com%7Ceef5f6c17aa24158479308d6f487a3bf%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C636965260739304265&amp;sdata=SFb9%2BvcnYqL9GfsFk7deZSJJZb6AhgYxctA0PUUijTQ%3D&amp;reserved=0
    
    
        The following commit(s) were added to refs/heads/develop by this push:
             new 9c2d5ca  Adding a resizable load inicator.
        9c2d5ca is described below
    
        commit 9c2d5ca53ef0c67950ff76753f5f2834568de90a
        Author: DESKTOP-RH4S838\Yishay <[email protected]>
        AuthorDate: Wed Jun 19 08:16:50 2019 +0300
    
            Adding a resizable load inicator.
        ---
         .../Basic/src/main/resources/basic-manifest.xml    |  1 +
         .../apache/royale/html/beads/DisableLoaderBead.as  |  6 +-
         .../html/beads/ShrinkableDisableLoaderBead.as      | 87 
++++++++++++++++++++++
         3 files changed, 91 insertions(+), 3 deletions(-)
    
        diff --git 
a/frameworks/projects/Basic/src/main/resources/basic-manifest.xml 
b/frameworks/projects/Basic/src/main/resources/basic-manifest.xml
        index a2239a7..284cf5c 100644
        --- a/frameworks/projects/Basic/src/main/resources/basic-manifest.xml
        +++ b/frameworks/projects/Basic/src/main/resources/basic-manifest.xml
        @@ -161,6 +161,7 @@
             <component id="DisableBead" 
class="org.apache.royale.html.beads.DisableBead" />
             <component id="DisabledAlphaBead" 
class="org.apache.royale.html.beads.DisabledAlphaBead" />
             <component id="DisableLoaderBead" 
class="org.apache.royale.html.beads.DisableLoaderBead" />
        +    <component id="ShrinkableDisableLoaderBead" 
class="org.apache.royale.html.beads.ShrinkableDisableLoaderBead" />
             <component id="DisableChildrenBead" 
class="org.apache.royale.html.beads.DisableChildrenBead" />
             <component id="NumericOnlyTextInputBead" 
class="org.apache.royale.html.accessories.NumericOnlyTextInputBead" />
             <component id="PasswordInputBead" 
class="org.apache.royale.html.accessories.PasswordInputBead" />
        diff --git 
a/frameworks/projects/Basic/src/main/royale/org/apache/royale/html/beads/DisableLoaderBead.as
 
b/frameworks/projects/Basic/src/main/royale/org/apache/royale/html/beads/DisableLoaderBead.as
        index a233bdf..19ef75b 100644
        --- 
a/frameworks/projects/Basic/src/main/royale/org/apache/royale/html/beads/DisableLoaderBead.as
        +++ 
b/frameworks/projects/Basic/src/main/royale/org/apache/royale/html/beads/DisableLoaderBead.as
        @@ -56,8 +56,8 @@ package org.apache.royale.html.beads
                     {
                     }
    
        -           private var _strand:IStrand;
        -           private var _loader:IUIBase;
        +           protected var _strand:IStrand;
        +           protected var _loader:IUIBase;
    
                     /**
                      *  @copy org.apache.royale.core.IBead#strand
        @@ -85,7 +85,7 @@ package org.apache.royale.html.beads
                 /**
                  * @royaleignorecoercion org.apache.royale.core.IUIBase
                  */
        -           private function get host():IUIBase
        +           protected function get host():IUIBase
                     {
                             return _strand as IUIBase;
                     }
        diff --git 
a/frameworks/projects/Basic/src/main/royale/org/apache/royale/html/beads/ShrinkableDisableLoaderBead.as
 
b/frameworks/projects/Basic/src/main/royale/org/apache/royale/html/beads/ShrinkableDisableLoaderBead.as
        new file mode 100644
        index 0000000..d0f697b
        --- /dev/null
        +++ 
b/frameworks/projects/Basic/src/main/royale/org/apache/royale/html/beads/ShrinkableDisableLoaderBead.as
        @@ -0,0 +1,87 @@
        
+////////////////////////////////////////////////////////////////////////////////
        +//
        +//  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
        +//
        +//      
https://nam04.safelinks.protection.outlook.com/?url=http%3A%2F%2Fwww.apache.org%2Flicenses%2FLICENSE-2.0&amp;data=02%7C01%7Caharui%40adobe.com%7Ceef5f6c17aa24158479308d6f487a3bf%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C636965260739304265&amp;sdata=tU2cTCNeBI0DCrtV51oUmgx7a02zIqH5SlbsB%2BDNLog%3D&amp;reserved=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.html.beads
        +{
        +   import org.apache.royale.html.LoadIndicator;
        +   import org.apache.royale.utils.PointUtils;
        +   import org.apache.royale.core.IPopUpHost;
        +   import org.apache.royale.utils.UIUtils;
        +   import org.apache.royale.geom.Point;
        +   import org.apache.royale.core.IUIBase;
        +   /**
        +    *  The ShrinkableDisableLoaderBead extends DisableLoaderBead to 
allow shrinking of the load indicator.
        +    *
        +    *  @langversion 3.0
        +    *  @playerversion Flash 10.2
        +    *  @playerversion AIR 2.6
        +    *  @productversion Royale 0.9.6
        +    */
        +   public class ShrinkableDisableLoaderBead extends DisableLoaderBead
        +   {
        +        private var _resizeFactor:Number = 1;
        +           /**
        +            *  constructor.
        +            *
        +            *  @langversion 3.0
        +            *  @playerversion Flash 10.2
        +            *  @playerversion AIR 2.6
        +            *  @productversion Royale 0.9.6
        +            */
        +           public function ShrinkableDisableLoaderBead()
        +           {
        +            super();
        +           }
        +
        +           /**
        +            *  The size of the load indicator relative to the strand.
        +         *  If it's 0.5 it's half the size.
        +            *
        +            *  @langversion 3.0
        +            *  @playerversion Flash 10.2
        +            *  @playerversion AIR 2.6
        +            *  @productversion Royale 0.9.6
        +            */
        +        public function get resizeFactor():Number
        +        {
        +            return _resizeFactor;
        +        }
        +
        +        public function set resizeFactor(value:Number):void
        +        {
        +            _resizeFactor = value;
        +        }
        +
        +           override protected function addLoadIndicator():void
        +           {
        +                   var point:Point = PointUtils.localToGlobal(new 
Point(0, 0), _strand);
        +                   _loader = new LoadIndicator();
        +                   _loader.width = (_strand as IUIBase).width * 
_resizeFactor;
        +                   _loader.height = (_strand as IUIBase).height * 
_resizeFactor;
        +                   _loader.x = point.x + (_strand as IUIBase).width / 
2 - _loader.width / 2;
        +                   _loader.y = point.y + (_strand as IUIBase).height / 
2 - _loader.height / 2;
        +                   COMPILE::JS
        +                   {
        +                           _loader.element.style.position = "absolute";
        +                   }
        +                   var popupHost:IPopUpHost = 
UIUtils.findPopUpHost(host);
        +                   popupHost.popUpParent.addElement(_loader);
        +           }
        +
        +   }
        +}
    
    
    
    

Reply via email to