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