[Laszlo-dev] For Review: Change 20081015-laszlo-2 Summary: add Method removeAllItems at Component baselist

[J Crowley]

Thanks!  And I'll fix that before checking it in.

Philip Romanik wrote:
> Approved, but you need to change the for loop slightly. Change
>
>                 for(var eg = 0; eg <= _contentview.subviews.length; eg++){
> to:
>                 for(var eg = 0; eg < _contentview.subviews.length; eg++){
>
>
>
>
> > Change 20081015-laszlo-2 by laszlo at T43-L3XEXMW on 2008-10-15 12:14:03
> > EDT in /home/laszlo/src/svn/openlaszlo/trunk-bigboss
> > for http://svn.openlaszlo.org/openlaszlo/trunk
> >
> > Summary: add Method removeAllItems at Component baselist
> >
> > New Features:
> >
> > Bugs Fixed: LPP-4054 - add Method removeAllItems at Component baselist
> >
> > Technical Reviewer: promanik
> > QA Reviewer: max
> > Doc Reviewer: (pending)
> >
> > Documentation:
> >
> > Release Notes:
> >
> > Details: Implemented a contributor fix, adding method "removeAllItems"
> > to component baselist. I sent this out for review last fall, but it
> > never got checked in.
> >
> > Tests: Run the following:
> >
> > <canvas width="500" height="300">
> >    <list name="list1">
> >        <textlistitem text="soln1" selected="false"/>
> >        <textlistitem text="soln2" selected="true"/>
> >        <textlistitem text="soln3" selected="false"/>
> >    </list>
> >    <button y="80" onclick="list1.removeAllItems()"/>
> > </canvas>
> >
> > Click the button.
> >
> > Files:
> > M lps/components/base/baselist.lzx
> >
> > Changeset:
> > http://svn.openlaszlo.org/openlaszlo/patches/20081015-laszlo-2.tar
> >
> >
> >
>