Approved, and thanks!

Note André's comment that LzConstraintExpr is a subclass of LzInitExpr, so you only have to test for the latter.

On 2008-02-14, at 22:56 EST, Max Carlson wrote:

Change 20080214-maxcarlson-B by [EMAIL PROTECTED] on 2008-02-14 19:19:09 PST
   in /Users/maxcarlson/openlaszlo/trunk
   for http://svn.openlaszlo.org/openlaszlo/trunk

Summary: Fix states and constrained text in DHTML, fix drawview

New Features:

Bugs Fixed: LPP-1587 'ECMA4: Compile LZX declarations as JS declarations' (partial)

Technical Reviewer: promanik
QA Reviewer: ptw
Doc Reviewer: (pending)

Documentation:

Release Notes:

Details: LzTextSprite.js - Don't store args.text from construct.

LzText.lzs - Default to '' in construct() when args.text instanceof LzConstraintExpr || args.text instanceof LzInitExpr

LzState.lzs - More safety checking when unpacking args.

drawview.lzx - Don't use args.height/width/clip to init drawview.

Tests: drawview works again, no warnings under IE when reloading pages. Constrained text works again.

Files:
M      WEB-INF/lps/lfc/kernel/dhtml/LzTextSprite.js
M      WEB-INF/lps/lfc/views/LzText.lzs
M      WEB-INF/lps/lfc/helpers/LzState.lzs
M      lps/components/extensions/drawview.lzx

Changeset: 
http://svn.openlaszlo.org/openlaszlo/patches/20080214-maxcarlson-B.tar


Reply via email to