Weird. What happens if you ask in the debugger on that system `(void 0) === null`? If it returns `true`, you might want to report a bug to Opera, because that's not the standard (and will break a lot of other stuff in the LFC). If it reports `false`, as it should, then something else is going on, and maybe we need more investigatin into the clipping issue in Opera.
On 2010-09-23, at 21:31, Max Carlson wrote: > 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 >>
