Should't it say: ... || subview is lz.layout) ?
I think that bit is required, so the layout doesn't get nuked when the menu is redrawn.
On 10/12/10 9:37 PM, Henry Minsky wrote:
Change hqm-20101012-saQ by [email protected] on 2010-10-12 23:35:20 EDT in /Users/hqm/openlaszlo/trunk-clean for http://svn.openlaszlo.org/openlaszlo/trunk Summary: fix for IE8 DHTML context menu bug New Features: Bugs Fixed: LPP-9434 Technical Reviewer: andre QA Reviewer: max Doc Reviewer: (pending) Documentation: Release Notes: Overview: Details: + remove reference to lzlayout in contextmenu.lzx#setItems, and rebuild contextmenu.js file Tests: test case from bug report, IE8/DHTML <canvas height="80%"> <button onclick="moveHoriz(5);"> Move<b>me</b> <method name="moveHoriz" args="moveAmount"> this.setAttribute("x", this.x+moveAmount); </method> </button> </canvas> Files: M WEB-INF/lps/lfc/contextmenu.js M WEB-INF/lps/lfc/contextmenu.lzx Changeset: http://svn.openlaszlo.org/openlaszlo/patches/hqm-20101012-saQ.tar
