On 5/25/10 7:54 AM, P T Withington wrote:
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.
Done.
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, ...);
I was thinking instead of doing this, I'd keep a hash of animators - so
we won't need to create a new one each time LzNode.animate() is called.
I'm going to save this for another change.
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
