Approved.

On 6/15/2009 11:04 PM, [email protected] wrote:
Change 20090615-Philip-0 by phi...@philip-dc on 2009-06-15 16:51:13 EDT
    in /cygdrive/f/laszlo/svn/src/svn/openlaszlo/trunk
    for http://svn.openlaszlo.org/openlaszlo/trunk

Summary: (REVISED) DHTML IE: Fix context menus on text

New Features:

Bugs Fixed:

Technical Reviewer: andre
QA Reviewer: max
Doc Reviewer: (pending)

Documentation:

Release Notes:

Details:
On the first call to document.elementFromPoint() in IE, it returns the scrolldiv created in LzTextSprite. As a result, the checks for context menu stopped (because the swf8mode checks were true).

The fix is to add another conditional to see if the returned element is the scrolldiv. If so, ignore it and document.elementFromPoint() will be called a second time to return the desired element. I added a quirk, ie_elementfrompoint.

This is still an issue for inputtext in dhtml (I tested with IE and FF). I created LPP-8268 to track this.


Tests:
See lpp-8254. Right clicking on text displays context menu in IE7/IE8. It continues to work in FF/Safari.

Files:
M      WEB-INF/lps/lfc/kernel/dhtml/LzSprite.js
M      WEB-INF/lps/lfc/kernel/dhtml/LzMouseKernel.js

Changeset: http://svn.openlaszlo.org/openlaszlo/patches/20090615-Philip-0.tar

Reply via email to