<html>
<body>
Approved!<br><br>
This example shows there is still something wonky going on when you try
and select the inputtext after moving between the modal dialog and the
canvas. It works much better than the current code!<br><br>
<br>
<canvas> <br>
<simplelayout axis="y"
spacing="5"/><br>
<inputtext width="100"
bgcolor="0xeeeeee"/> <br>
<button onclick="md.open();">Show Modal
Dialog</button> <br>
<modaldialog id="md" width="200"
height="100" title="Modal Dialog"> <br>
<simplelayout axis="y"
spacing="5"/><br>
<inputtext width="100"
bgcolor="0xeeeeee"/> <br>
<button
onclick="parent.close()">OK</button> <br>
</modaldialog> <br>
</canvas> <br><br>
<br>
<blockquote type=cite class=cite cite="">The testcase at the top of
LPP-4916 works for me in IE 6 and 7. Approved!<br>
<br>
André Bargull wrote:<br>
> Ok, here is my next/last attempt to fix LPP-4916: <br>
>
<a href="http://svn.openlaszlo.org/openlaszlo/patches/20071115-bargull-4.tar">
http://svn.openlaszlo.org/openlaszlo/patches/20071115-bargull-4.tar</a>
<br>
> This time, I tried to bind __LZclick's display-style on the
visibility <br>
> of a sprite, in belief this may help to get around the LPP-3307
issues..<br>
> (dhtml/IE6 is working and dhtml/IE7 isn't working, but ie7 should
<br>
> amend ie6, shouldn't it?)<br>
> <br>
> If this change isn't working either [under dhtml/IE7], I guess Max
<br>
> should give some input on the changes for LPP-3307, especially on
<br>
> "LzSprite.prototype.setVisible(..)".<br>
> <br>
> This is the current (erroneous) state of<br>
> "LzSprite.prototype.setVisible(..)":<br>
> ---<br>
> [...]<br>
> if (this.__LZclick && this.quirks.fix_ie_clickable) {<br>
> /* we won't change __LZclickdiv's
display-style<br>
> * as soon as we have got a __LZclick-div
(ie-only)<br>
> * => buggy!<br>
> */<br>
> this.__LZclick.style.display = v ? '' : 'none'; }
else {<br>
> this.__LZclickdiv.style.display = v ? 'block' :
'none'; } [...]<br>
> ---<br>
> <br>
> On 12/2/2007 4:43 AM, Philip Romanik wrote:<br>
>> I tried your new changeset but I'm seeing the same behavior in
IE7/dhtml.<br>
>> <br>
>> <br>
>> <br>
>>> The application from LPP-3307 is working for me in IE6/dhtml
with <br>
>>> the previous changeset...<br>
>>> May you try this one (updated):<br>
>>>
<a href="http://svn.openlaszlo.org/openlaszlo/patches/20071115-bargull-4.tar">
http://svn.openlaszlo.org/openlaszlo/patches/20071115-bargull-4.tar</a>
<br>
>>> <br>
>>> - André<br>
>>> <br>
>>> On 12/1/2007 7:40 PM, Philip Romanik wrote:<br>
>>> > The change fixes LPP-3539, but it breaks the app from
LPP-3307 (as <br>
>>> > well as the weather app) in IE7/dhtml.<br>
>>> ><br>
>>> > The app from LPP-3307 is:<br>
>>> ><br>
>>> > <canvas height="200"><br>
>>> > <view clickable="true"
width="300" height="200"><br>
>>> > <inputtext x="50"
width="175" bgcolor="0xeeeeee"/><br>
>>> > </view><br>
>>> > </canvas><br>
>>> ><br>
>>> > In IE7/dhtml the inputtext cannot be selected. If I
remove the <br>
>>> > changeset, LPP-3307 works again.<br>
>>> ><br>
>>> ><br>
>>> ><br>
>>> ><br>
>>> ><br>
>>> >> Still waiting for your reviews.<br>
>>> >> [This change resolves LPP-3539, LPP-4268, LPP-4916,
LPP-4919 and <br>
>>> >> LPP-5174.]<br>
>>> >> >> -------- Original-Message --------<br>
>>> >> Subject: For Review: Change
20071115-bargull-4 Summary: clickable-<br>
>>> >> and<br>
>>> >> visible-property in IE<br>
>>> >> Date: Thu, 15
Nov 2007 23:36:11 +0100<br>
>>> >> From: André
Bargull <a.bargull@intensis.de><br>
>>> >> To: max@laszlosystems.com,
promanik@laszlosystems.com<br>
>>> >> CC: laszlo-dev@openlaszlo.org<br>
>>> >> >> >> >> Change
20071115-bargull-4 by bargull@dell--p4--2-53 on<br>
>>> 2007-11-15<br>
>>> >> 23:20:38 in
/home/Admin/src/svn/openlaszlo/trunk<br>
>>> >> for
<a href="http://svn.openlaszlo.org/openlaszlo/trunk" eudora="autourl">
http://svn.openlaszlo.org/openlaszlo/trunk</a><br>
>>> >> >> Summary: clickable- and visible-property
in IE New Features:<br>
>>> >> >> Bugs Fixed:<br>
>>> >> LPP-4916 - modaldialog cannot be dragged w/ IE7
DHTML in trunk <br>
>>> >> build<br>
>>> >> 6836<br>
>>> >> >> Technical Reviewer: max<br>
>>> >> QA Reviewer: promanik<br>
>>> >> Doc Reviewer: (pending)<br>
>>> >> >> Documentation:<br>
>>> >> >> Release Notes:<br>
>>> >> >> Details:<br>
>>> >> Reverting parts of changesets rev. #3457 and rev.
#3460 to fix<br>
>>> LPP-4916.<br>
>>> >> These changesets seems to be unnecessary (at least
now), because <br>
>>> >> removing it does not revive the bug at LPP-3307
(Max might give <br>
>>> >> more infos to LPP-3307).<br>
>>> >> >> >> Tests:<br>
>>> >> see testcases at bugreport<br>
>>> >> >> Files:<br>
>>> >> M WEB-INF/lps/lfc/kernel/dhtml/LzSprite.js<br>
>>> >> >> Changeset:<br>
>>> >>
<a href="http://svn.openlaszlo.org/openlaszlo/patches/20071115-bargull-4.t">
http://svn.openlaszlo.org/openlaszlo/patches/20071115-bargull-4.t</a><br>
>>> >> ar<br>
>>> <br>
> <br>
<br>
--<br>
Regards,<br>
Max Carlson<br>
OpenLaszlo.org</blockquote></body>
</html>