saket / InboxRecyclerView

Build expandable descendant navigation, inspired by Google Inbox

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Issues with InboxRecyclerView inside NestedScrollView

andyburris opened this issue · comments

When the InboxRecyclerView is inside a NestedScrollView, there are lots of issues, such as the ExpandablePage expanding from farther down the list than wanted, and the items all being pushed down and replaced with blanks. Not sure if it's inherent to the NestedScrollView but it wasn't happening with regular RecyclerView.

@andb3 this will be fun to debug at. Can I ask you to fork this project and update the sample app to reproduce this problem?

Ok, it's up at https://github.com/andb3/InboxRecyclerViewSample and the issue reproduces on the new sample

Thanks @andb3. I'll try taking a look this weekend.

Hey @saket, I made some changes in my local repository and I think I got it working with a custom ItemExpandAnimator.
The only difference otherwise is some added logic in the expandItem and alignItemWithExpandingItem functions to see if there is scroll offset needed due to the NestedScrollView.
If you want to take a look, it's in the fix branch in my repository

I haven't been able to take a look at this. Considering that you've already found a fix, do you want to send a PR?

Sure, let me fix a few issues I've found and I'll make one

Sure 👍

please fix the bug: inbox for androidx.core.widget.NestedScrollView ,thx!

A bit late, but this should be solved with v3.0.0 that brings support for nested scrolling: https://github.com/saket/InboxRecyclerView/releases/tag/v3.0