Comments:
1) I'd just remove all the animator functions you commented out of LzNode.
They are clutter and will (already for one of them) rot. svn can find them if
we really need them back.
2) Rather than creating multiple object hashes for the various animation
attributes, why not define a class (data structure) that holds all the
animation properties of an attribute? Thus, when animating an attribute, you
will just say something like:
var aas = this.__animatedAttributes;
if (! aas[name]) { aas[name] = new AnimationAttribute(start, end, ...);
?
On 2010-05-24, at 20:20, Max Carlson wrote:
> Change 20100524-maxcarlson-q by maxcarl...@friendly on 2010-05-24 17:18:32 PDT
> in /Users/maxcarlson/openlaszlo/trunk-clean
> for http://svn.openlaszlo.org/openlaszlo/trunk
>
> Summary: Clean up animator/node interaction
>
> Bugs Fixed: LPP-9020 - Support CSS3 transitions (partial)
>
> Technical Reviewer: ptw
> QA Reviewer: hminsky
>
> Details: LzNode - Comment out simplified versions of animator methods now
> that they're inlined in LaszloAnimation.
>
> LaszloAnimation - Shorten lookpus, inline calls to LzNode for clarity,
> performance.
>
> Tests: Animators work as before across runtimes
>
> Files:
> M WEB-INF/lps/lfc/core/LzNode.lzs
> M WEB-INF/lps/lfc/controllers/LaszloAnimation.lzs
>
> Changeset:
> http://svn.openlaszlo.org/openlaszlo/patches/20100524-maxcarlson-q.tar
