I fixed this even though (strangely) it seemed to work fine in Opera 10.62 on Windows 7. Thanks for pointing out the error!

On 9/23/10 3:39 PM, P T Withington wrote:
Are you sure about this?  If the copyclip arg is omitted, it will be undefined 
(aka (void 0)), and:

js>  (void 0) === null
false
js>

On 2010-09-23, at 16:22, [email protected] wrote:

Author: max
Date: 2010-09-23 13:22:09 -0700 (Thu, 23 Sep 2010)
New Revision: 17584

Modified:
   openlaszlo/trunk/WEB-INF/lps/lfc/kernel/dhtml/LzSprite.js
Log:
Change maxcarlson-20100923-fGM by maxcarl...@friendly on 2010-09-23 13:18:12 PDT
    in /Users/maxcarlson/openlaszlo/trunk-clean
    for http://svn.openlaszlo.org/openlaszlo/trunk

Summary: Fix clickdivs/clipping in DHTML Opera 10.6

Bugs Fixed: LPP-9380,  LPP-9382

Technical Reviewer: hminsky

Details: Need to test explicitly for null copyclip argument in __copystyles()

Tests: See LPP-9380, LPP-9382



Modified: openlaszlo/trunk/WEB-INF/lps/lfc/kernel/dhtml/LzSprite.js
===================================================================
--- openlaszlo/trunk/WEB-INF/lps/lfc/kernel/dhtml/LzSprite.js   2010-09-23 
20:21:49 UTC (rev 17583)
+++ openlaszlo/trunk/WEB-INF/lps/lfc/kernel/dhtml/LzSprite.js   2010-09-23 
20:22:09 UTC (rev 17584)
@@ -3011,7 +3011,7 @@
   * @access private
   */
LzSprite.prototype.__copystyles = function(from, to, copyclip) {
-    copyclip = copyclip || true;
+    copyclip = copyclip === null ? true : copyclip;
     var sprite = from.owner;
     var left = sprite._x;
     if (left) {


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

Reply via email to