Comments:

You should leave the arglist to `initialize` empty, since you don't use any args. The list you give is the arglist of subclasses of LzNode, not relevant here.

Not sure I understand how __LZtrackDel is called on idle, since it does not register itself. Is it somehow hardwired? That seems like bad modularity. Perhaps this is a question for Max.

Otherwise approved!

On 2007-05-29, at 14:32 EDT, Philip Romanik wrote:

Change 20070529-Philip-6 by [EMAIL PROTECTED] on 2007-05-29 14:15:49 EST
    in /cygdrive/f/laszlo/svn/src/svn/openlaszlo/branches/legals
    for http://svn.openlaszlo.org/openlaszlo/branches/legals

Summary: Ported LzTrack.lzs to new class system

New Features:

Bugs Fixed: LPP-3986

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

Documentation:

Release Notes:

Details:
Ported LzTrack.lzx to the new class system. LzTrack is a singleton class but is written more like an ordinary class. To minimize the changes I made (and to make it 100% compatible with the old version), the class name is LzTrackClass. At the bottom of the file is:

    /** The single instance of this class
      * @access private
      */
    var LzTrack = new LzTrackClass;



Tests:
lzpix in swf and dhtml. The best test in this app is to drag an image into the clip region. When I first ported LzTrack the image disappeared when I released the mouse.

Run these in swf and dhtml: http://localhost:8080/legals/test/lfc/ lztrack/trackgroups.lzx
http://localhost:8080/legals/test/lfc/lztrack/trackgroup.lzx

Files:
M      WEB-INF/lps/lfc/services/LzTrack.lzs

Changeset: http://svn.openlaszlo.org/openlaszlo/patches/20070529- Philip-6.tar



Reply via email to