Not approved yet:

Right now, LzText seems to think it can tell the sprite to change the font and 
then immediately update its size by querying the sprite.  Clearly that does not 
work in all cases.  With this change, the LFC is still going to try to update 
the size synchonously, and then the sprite will also install a callback to 
update the size asynchronously.  Do we really want this redundancy? It seems 
inefficient if not outright wrong.

Why don't we amend the LFC/Kernel API to say the LFC tells the sprite what to 
do and then the sprite will call back to the LFC if there is a size change?
 
On 2010-03-25, at 13:47, Max Carlson wrote:

> Change 20100325-maxcarlson-k by maxcarl...@bank on 2010-03-25 10:44:20 PDT
>    in /Users/maxcarlson/openlaszlo/trunk-clean
>    for http://svn.openlaszlo.org/openlaszlo/trunk
> 
> Summary: Update viewsystem size when compiled fonts load in DHTML
> 
> Bugs Fixed: LPP-8848 - DHTML: Textfields using embedded fonts aren't telling 
> the viewsystem when their size changes
> 
> Technical Reviewer: ptw
> QA Reviewer: hminsky
> 
> Details: Call _updateSize() on the owner/view.
> 
> Tests: See LPP-8848
> 
> Files:
> M       WEB-INF/lps/lfc/kernel/dhtml/LzTextSprite.js
> 
> Changeset: 
> http://svn.openlaszlo.org/openlaszlo/patches/20100325-maxcarlson-k.tar
> 


Reply via email to