<layout> does not extend <view>, so there won't be any <layout>
instances in container.subviews, cf. <view>#subviews,
<view>#addSubview(..), <view>#__LZinstantiationDone().
On 10/13/2010 6:36 AM, Max Carlson wrote:
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
