incubator-flex-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Alex Harui <aha...@adobe.com>
Subject Re: svn commit: r1423863 - /incubator/flex/sdk/branches/develop/frameworks/projects/charts/src/mx/charts/DateTimeAxis.as
Date Wed, 19 Dec 2012 22:47:44 GMT
Ok.  I didn't see that.


On 12/19/12 2:45 PM, "Chema Balsas" <jbalsas@gmail.com> wrote:

>> 
>> I don't think that's the right change.  I'm guessing the intent is to
>> change
>> a local variable, not the instance's height property?
> 
> 
> I think height was already a local variable as it was being passed as an
> argument to the call
> 
>     override protected function drawRowBackground(s:Sprite, rowIndex:int,
>>                                          y:Number, height:Number,
>> color:uint, dataIndex:int):void
> 
> 
> 2012/12/19 Alex Harui <aharui@adobe.com>
> 
>> 
>> 
>> 
>> On 12/19/12 2:23 PM, "Chema Balsas" <jbalsas@gmail.com> wrote:
>> 
>>>> 
>>>> But but what about this
>>>> Was:
>>>> var height:Number = Math.min(height,
>>>>         listContent.height -
>>>>         y);
>>>> changed to:
>>>>         height = Math.min(height, listContent.height - y);
>> Hi Chema,
>> 
>> I don't think that's the right change.  I'm guessing the intent is to
>> change
>> a local variable, not the instance's height property?
>> 
>> --
>> Alex Harui
>> Flex SDK Team
>> Adobe Systems, Inc.
>> http://blogs.adobe.com/aharui
>> 
>> 

-- 
Alex Harui
Flex SDK Team
Adobe Systems, Inc.
http://blogs.adobe.com/aharui


Mime
View raw message