Approved! You may want to do a test for inputtext to see if text behind
it interferes with context menus...
[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
--
Regards,
Max Carlson
OpenLaszlo.org
