Author: jcrowley
Date: 2007-11-15 02:12:32 -0800 (Thu, 15 Nov 2007)
New Revision: 7287

Modified:
   openlaszlo/trunk/WEB-INF/lps/lfc/views/LaszloView.lzs
Log:
Change 20071114-jcrowley-H by [EMAIL PROTECTED] on 2007-11-14 13:48:11 EST
    in /Users/jcrowley/src/svn/openlaszlo/trunk-g
    for http://svn.openlaszlo.org/openlaszlo/trunk

Summary: valign and y attribute should have warning

New Features:

Bugs Fixed: LPP-2743 - valign and y attribute should have warning

Technical Reviewer: max
QA Reviewer: pbr
Doc Reviewer: 

Documentation:

Release Notes:

Details: Added warnings if x and align or y and
        valign are both used on a view, since
        align and valign supersede x and y.

Tests: Run the following in SWF and DHTML:  Note it
        produces a warning if x and align or y and
        valign are declared simultaneously.  Does not
        warn if either is used individually.

<canvas>
    <view x="10" align="center"/>
    <view y="10" valign="middle"/>
    <view x="10"/>
    <view y="10"/>
    <view align="center"/>
    <view valign="middle"/>
</canvas>



Modified: openlaszlo/trunk/WEB-INF/lps/lfc/views/LaszloView.lzs
===================================================================
--- openlaszlo/trunk/WEB-INF/lps/lfc/views/LaszloView.lzs       2007-11-15 
10:03:01 UTC (rev 7286)
+++ openlaszlo/trunk/WEB-INF/lps/lfc/views/LaszloView.lzs       2007-11-15 
10:12:32 UTC (rev 7287)
@@ -308,6 +308,14 @@
     if ( r  != null ){
         if (this.sprite) this.setResource( r );
     }
+    if($debug){
+        if( 'valign' in args && args['valign'] && 'y' in args && args['y']){
+            Debug.warn("y attribute ignored; superseded by valign 
constraint.");
+        }
+        if( 'align' in args && args['align'] && 'x' in args && args['x']){
+            Debug.warn("x attribute ignored; superseded by align constraint.");
+        }
+    }
 }
 
 /** Receive attribute/event change from sprite


_______________________________________________
Laszlo-checkins mailing list
[email protected]
http://www.openlaszlo.org/mailman/listinfo/laszlo-checkins

Reply via email to