[Laszlo-checkins] r16826 - openlaszlo/trunk/WEB-INF/lps/lfc/kernel/dhtml

[max@openlaszlo.org]

Author: max
Date: 2010-06-28 21:45:33 -0700 (Mon, 28 Jun 2010)
New Revision: 16826

Modified:
   openlaszlo/trunk/WEB-INF/lps/lfc/kernel/dhtml/LzSprite.js
Log:
Change 20100628-maxcarlson-F by maxcarlson at friendly on 2010-06-28 12:44:18 PDT
    in /Users/maxcarlson/openlaszlo/trunk-clean
    for http://svn.openlaszlo.org/openlaszlo/trunk

Summary: Fix spurious mouseover events in IE DHTML

Bugs Fixed: LPP-9107 - DHTML - IE7 - Amazon - When drag the album, the frame of album will be stay at original place

Technical Reviewer: hminsky
QA Reviewer: ptw

Details: IE needs an extra check to make sure the mosue is still over __lastMouseOver before sending an onmosueover event.

Tests: See LPP-9107, testcase at LPP-8445 runs as before



Modified: openlaszlo/trunk/WEB-INF/lps/lfc/kernel/dhtml/LzSprite.js
===================================================================
--- openlaszlo/trunk/WEB-INF/lps/lfc/kernel/dhtml/LzSprite.js	2010-06-29 03:22:36 UTC (rev 16825)
+++ openlaszlo/trunk/WEB-INF/lps/lfc/kernel/dhtml/LzSprite.js	2010-06-29 04:45:33 UTC (rev 16826)
@@ -1672,7 +1672,18 @@
     //Debug.info('__globalmouseup', LzMouseKernel.__lastMouseOver, e);
     if (LzMouseKernel.__lastMouseOver) {
         // send artificial onmouseover event - see LPP-8445
-        LzMouseKernel.__lastMouseOver.__mouseEvent('onmouseover', true);
+        var sendmouseover = true;
+        // the mouse went up on this sprite
+        if (this.quirks.ie_mouse_events) {
+            // cancel sending if the mouse isn't currently over __lastMouseOver
+            // see LPP-9107
+            if (! LzMouseKernel.__lastMouseOver.__isMouseOver()) {
+                sendmouseover = false;
+            }
+        }
+        if (sendmouseover) {
+            LzMouseKernel.__lastMouseOver.__mouseEvent('onmouseover', true);
+        }
         LzMouseKernel.__lastMouseOver = null;
     }
 }