Approved, but you need to make a change. I see warnings in swf when I
select something from the list and click on the edittextbox. You need to
check that the variable exists before accessing it. I used ('foo' in this)
to check:
<method name="destroy" ><![CDATA[
if ('_mwActivateDel' in this) {
this._mwActivateDel.unregisterAll();
delete this._mwActivateDel;
}
if ('_mwDeactivateDel' in this) {
this._mwDeactivateDel.unregisterAll();
delete this._mwDeactivateDel;
}
if ('_mwUpdateDel' in this) {
this._mwUpdateDel.unregisterAll();
delete this._mwUpdateDel;
}
if ('clipSizeDel' in this) {
this.clipSizeDel.unregisterAll();
delete this.clipSizeDel;
}
if ('targetHeightDel' in this) {
this.targetHeightDel.unregisterAll();
delete this.targetHeightDel;
}
if ('targetPosDel' in this) {
this.targetPosDel.unregisterAll();
delete this.targetPosDel;
}
if ('heightDel' in this) {
this.heightDel.unregisterAll();
delete this.heightDel;
}
delete this.scrolltarget;
delete this.focusview;
delete this.othersb;
super.destroy.apply(this, arguments);
]]></method>
As indicated, this also fixes LPP-4526
Change 20070819-maxcarlson-g by [EMAIL PROTECTED] on 2007-08-19
10:28:24 PDT
in /Users/maxcarlson/openlaszlo/wafflecone
for http://svn.openlaszlo.org/openlaszlo/branches/wafflecone
Summary: Clean-up for "basescrollbar"
New Features:
Bugs Fixed: LPP-4427 - Clean-up for "basescrollbar"
Technical Reviewer: promanik
QA Reviewer: jcrowley
Doc Reviewer: (pending)
Documentation:
Release Notes:
Details: Patched per LPP-4427.
Tests: See LPP-4427.
Files:
M lps/components/base/basescrollbar.lzx
Changeset: http://svn.openlaszlo.org/openlaszlo/patches/20070819-
maxcarlson-g.tar
