Well certainly you could not call getTextWidth 3 times.  If the text  
element is resizable, I believe that when you set the text it will  
have its .width set.  You could verify that .width == getTextWidth,  
and if so, replace the call with an accessor.

It is possible that getTextWidth is from days gone by...

On 2006-09-30, at 21:14 EDT, Benjamin Shine wrote:

> Calendar does a whole bunch of text measurement. For instance in the
> monthtitle button:
>                  <method event="ontext">
>                      this.setX( Math.round( ( this.parent.width -
>                                 this.getTextWidth())/2.0) - 2);
>                      this.setWidth (this.getTextWidth()+2);
>                      this.parent.monthtitleshdw.setWidth
> (this.getTextWidth()+2);
>
>                  </method>
> Can this all be replaced with something simpler and faster?
>
> On Sep 29, 2006, at 4:00 PM, Max Carlson wrote:
>
>> Comments:
>> * There is no x/yscaling in DHTML...
>>
>> * _x/yscale are flash-specific properties - you probably want to
>> read it out of the sprite as it'll be undefined here:
>> +    var scale = "_" + xory + "scale";
>>
>> * These should use kernel API calls instead of if ($as2):
>> + if ($as2) {
>> +        if ((a.sprite.__LZmovieClipRef[scale] / 100) != a[scale]) {
>> +          Debug.error("%s != %s", a.sprite.__LZmovieClipRef
>> [scale] / 100, a[scale]);
>> +        }
>> +      }
>> ..
>> +      if ($as2) {
>> +        if ((a.sprite.__LZmovieClipRef[scale] / 100) != a[scale]) {
>> +          Debug.error("%s != %s", a.sprite.__LZmovieClipRef
>> [scale] / 100, a[scale]);
>> +        }
>> +      }
>>
>> -Max
>>
>> P T Withington wrote:
>>> Change change.C6vFh0Zaf.txt by [EMAIL PROTECTED] /Users/ptw/
>>> pending-changes/ on 2006-09-29 18:19:09 EDT
>>> Summary: Port LvViewLinkage to DHTML
>>> Bugs Fixed:
>>> LPP-2816 getAttributeRelative inside a constraint doesn't work in
>>> dhtml
>>> Technical Reviewer: adam, max (pending)
>>> QA Reviewer: ben (pending)
>>> Doc Reviewer: n/a
>>> Details:
>>>     Library: LzObjects -> LzViewLinkage
>>>     LzSprite, LaszloView: move getLinkage back to View.
>>>     LzViewLinkage: Remove obsolete Object.error.  Put in class
>>>     declaration.  Simplify loop, use portable _*scale rather than
>>>     movieClip.
>>> Tests:
>>>     Test filed with bug works in both swf and dhtml.
>>>     QA reviewer should verify that it fixes the original problem
>>> with calendar.
>>> Files:
>>> D      kernel/swf/LaszloObjects.as
>>> M      kernel/swf/Library.lzs
>>> M      kernel/swf/LzSprite.as
>>> A  +   views/LzViewLinkage.lzs
>>> M      views/LaszloView.lzs
>>> M      views/Library.lzs
>
>
> _______________________________________________
> Laszlo-dev mailing list
> [email protected]
> http://www.openlaszlo.org/mailman/listinfo/laszlo-dev


_______________________________________________
Laszlo-dev mailing list
[email protected]
http://www.openlaszlo.org/mailman/listinfo/laszlo-dev

Reply via email to