Understand. But Phil had already TODO-ed this against the modules bug.
Surely you are not going to say we can't have runtime-specific
includes, are you? Do I need to make an LzDebugSprite?
On 2006-10-05, at 18:38 EDT, Max Carlson wrote:
> Fair enough - I'm just continuing my crusade against runtime-
> specific assertions in the LFC :)
>
> -Max
>
> P T Withington wrote:
>> 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
