The real fix is for us to have modules and for global to be a module,
not to be === _root/window. There is no reason for this to be either
a kernel API or try/catch.
On 2006-10-05, at 17:28 EDT, Max Carlson wrote:
> Can I ask that you add a TODO comment to change this to a kernel
> API or plain old try/catch when the SWF backend works?
>
> -Max
>
> Philip Romanik wrote:
>> Hi Tucker,
>> Your version works fine on swf/FF dhtml/IE6 dhtml. I'll check it
>> in shortly. Here's a simple app to tickle the issue on IE6 dhtml:
>> <canvas>
>> <text id="tempview" initstage="early">tempview</text>
>> <method reference="canvas" event="oninit">
>> tempview.destroy();
>> </method>
>> </canvas>
>> Phil
>>> Approved, but...
>>>
>>> Could you see if this works:
>>>
>>> if ($dhtml) {
>>> try {
>>> delete global[this.id];
>>> } catch (e) {
>>> // If you can't delete, set to undefined
>>> global[this.id] = void 0;
>>> }
>>> } else {
>>> delete global[this.id];
>>> }
>>>
>>> If it does, use that instead.
>>>
>>> On 2006-10-05, at 11:03 EDT, Philip Romanik wrote:
>>>
>>>> Change change.MJyDT2300.txt by [EMAIL PROTECTED] /cygdrive/f/laszlo/
>>>> svn/src/svn/openlaszlo/branches/legals/WEB-INF/lps/lfc/ on
>>>> 2006-10-05 10:55:00 EDT
>>>>
>>>> Summary: Don't delete property from global.
>>>>
>>>> New Features:
>>>>
>>>> Bugs Fixed:
>>>>
>>>> Technical Reviewer: ptw
>>>> QA Reviewer: (pending)
>>>> Doc Reviewer: (pending)
>>>>
>>>> Documentation:
>>>>
>>>> Release Notes:
>>>>
>>>> Details:
>>>> On IE6, you cannot delete a global property. The LzNode.destroy()
>>>> method does this. The right way to fix this is to do LPP-1871. In
>>>> the meantime, instead of deleting the property, I set it to null.
>>>> There is a comment in the code that references LPP-1871
>>>>
>>>>
>>>> Tests:
>>>> calendar view loads on IE6.
>>>>
>>>> Files:
>>>> M core\LzNode.lzs
>>>> <patch.Philip.4556.tgz>
>> _______________________________________________
>> Laszlo-dev mailing list
>> [email protected]
>> http://www.openlaszlo.org/mailman/listinfo/laszlo-dev
_______________________________________________
Laszlo-dev mailing list
[email protected]
http://www.openlaszlo.org/mailman/listinfo/laszlo-dev
