airbnb / epoxy

Epoxy is an Android library for building complex screens in a RecyclerView
https://goo.gl/eIK82p
Apache License 2.0
8.53k stars 728 forks source link

Make StickyHeaderLinearLayoutManager work with any RecyclerView.Adapter that implements StickyHeaderCallbacks #1037

Open optisamit opened 4 years ago

optisamit commented 4 years ago

Right now if I want to have sticky headers, I'm forced to use BaseEpoxyAdapter. But that does not need to be the case, since StickyHeaderLinearLayoutManager only needs an adapter implementing StickyHeaderCallbacks. Implementing this change would make the layout manager more flexible.

elihart commented 4 years ago

Are you saying you want to use the sticky headers outside of Epoxy?

optisamit commented 4 years ago

Yes. Why not?

On Thu, Sep 3, 2020, 02:35 Eli Hart notifications@github.com wrote:

Are you saying you want to use the sticky headers outside of Epoxy?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/airbnb/epoxy/issues/1037#issuecomment-686101953, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACMZRU6FWAPBDCCP2TCQLCLSD3JCXANCNFSM4P7V5UQA .

elihart commented 4 years ago

Because this is the Epoxy library, why not use Epoxy if you are here?

If you are not using Epoxy it seems preferrable to use a standalone sticky header library (like https://github.com/qiujayen/sticky-layoutmanager) which would probably offer better support and features)

optisamit commented 4 years ago

The library you've referenced has been abandoned for 3 years. I couldn't find anything better than Epoxy, but I wouldn't like to rewrite my adapters and view holders at this point

On Fri, Sep 4, 2020, 01:23 Eli Hart notifications@github.com wrote:

Because this is the Epoxy library, why not use Epoxy if you are here?

If you are not using Epoxy it seems preferrable to use a standalone sticky header library (like https://github.com/qiujayen/sticky-layoutmanager) which would probably offer better support and features)

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/airbnb/epoxy/issues/1037#issuecomment-686794463, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACMZRU5EBHBB4IMSXTJG5HLSEAJN7ANCNFSM4P7V5UQA .

Gelacio commented 1 year ago

Any plan to implement 'StickyHeaderGridLayoutManager' to support more than a span laid out? Let's say one span for the sticky item and then I can specify more than one span for the rest of the items?