That change caught my eye when I saw the commit email but I figured the
tests would catch any problem, which I guess it did.

I haven't looked at the code in detail, but I think addLocation is used
when single items are added.  I think some components may think if they
get an add at row 3 that rows 0-2 don't need updating, etc.

-Alex

On 2/25/14 1:25 PM, "Justin Mclean" <jmcl...@apache.org> wrote:

>Hi,
>
>Problem sorted it was the change to ListCollectionView which mean we may
>still have a regression issue (form 4.10). Can anyone lend an hand with
>this?
>
>Change was line 1211 from this:
>        if (localIndex && addedItems.length > 1)
>        {
>            addLocation = -1;
>        }
>to:
>        if (localIndex && addedItems.length > 0)
>        {
>            addLocation = -1;
>        }
>
>Thanks,
>Justin

Reply via email to