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

Reply via email to