[Laszlo-checkins] r11011 - openlaszlo/trunk/lps/components/base

[bargull@openlaszlo.org]

Author: bargull
Date: 2008-09-15 23:45:54 -0700 (Mon, 15 Sep 2008)
New Revision: 11011

Modified:
   openlaszlo/trunk/lps/components/base/basewindow.lzx
Log:
Change 20080913-bargull-vJR by bargull at dell--p4--2-53 on 2008-09-13 17:26:33
    in /home/Admin/src/svn/openlaszlo/trunk
    for http://svn.openlaszlo.org/openlaszlo/trunk

Summary: replace handler with setter

New Features:

Bugs Fixed: LPP-6747

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

Documentation:

Release Notes:

Details:
The "basewindow#setVisible"-method was transformed to a handler when we had no setter-tag, now we can revert that workaround.
    

Tests:



Modified: openlaszlo/trunk/lps/components/base/basewindow.lzx
===================================================================
--- openlaszlo/trunk/lps/components/base/basewindow.lzx	2008-09-16 03:47:24 UTC (rev 11010)
+++ openlaszlo/trunk/lps/components/base/basewindow.lzx	2008-09-16 06:45:54 UTC (rev 11011)
@@ -106,7 +106,7 @@
           if (this.mousedel) {
               this.mousedel.unregisterAll();
           }
-          super.destroy.apply(this, arguments);
+          super.destroy();
         </method>
         
         <method name="sendInFrontOf" args="v"> <![CDATA[
@@ -246,7 +246,8 @@
               front.
               @param Boolean isVisible: whether the window should be made
               visible. -->
-        <handler name="onvisible" args="isVisible"> <![CDATA[
+        <setter name="visible" args="isVisible"> <![CDATA[
+            super.setAttribute('visible', isVisible);
             if (isVisible) {
                 if (this.isinited) {
                     //need to add this window to the parent's windowlist
@@ -274,7 +275,7 @@
                 }
             }
             ]]>
-        </handler>
+        </setter>
 
         <!--- @keywords private -->
         <attribute name="mousedel" value="null"/>