Collection Repeat With Search Input

[quilligana]

See my post on the forum which outlines an issue with re-rendering a collection-repeat list following a filter being applied through a search input. If I first scroll down the list and then enter the search term, the new filtered list is not scrolled to the top. You can reproduce in this codepen. The items are all there and present but the scroll view doesn't seem to resize. If you delete the search input text and re-enter again, it all works fine as the scroll has been reset to the top of the list.

On a project I was working on, I implemented a temporary fix by adding scrollView.scrollTo(0,0, false, null, true); to the end of the renderer(value) function within the collectionRepeat directive in ionic.bundle.js.

I'm not sure if this is the best way to resolve this issue but if someone wants to advise/confirm the best solution, I will happily create a PR with a fix.

[ffalcinelli]

The workaround I was using in beta13 is to add ng-change="scrollTop()" to the input field.
But with the current master the focus is stolen from the input when the list scrolls to top, so it's no more an option I guess.

[joshbuchea]

This works great! Thanks @quilligana.

[joshbuchea]

@quilligana thanks for your efforts with this issue.

This commit solves the issue of the scrollview not resizing and items in a newly filtered list not being visible, however, it presents a new issue for me: now whenever I trigger a popover or modal to open from the view containing the list, it resets the list (scrolls to top).

I'm trying to get the list to behave properly while being filtered, AND also not lose scroll position when a modal or popup is displayed. Anyone have an idea how to solve this? Seems like maybe another check in the render function could solve this..? Any help or leads would be greatly appreciated. Thanks!

[ajoslin]

Fixed by just making the scrollView resize when data changes. Thanks for showing the bug and explaining your solution, it helped me find this one.