Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

期望同时支持列表上下分页预加载 #114

Closed
CosmicRayLucky opened this issue Mar 1, 2022 · 7 comments
Closed

期望同时支持列表上下分页预加载 #114

CosmicRayLucky opened this issue Mar 1, 2022 · 7 comments
Labels
寻求帮助 Extra attention is needed

Comments

@CosmicRayLucky
Copy link

问题描述

看代码，当前的预加载跟预拉取是互斥的，预拉取只是把View反转，并不能向上向下加载共存

期望行为

预加载、预拉取可以共存
@CosmicRayLucky CosmicRayLucky added the 寻求帮助 Extra attention is needed label Mar 1, 2022
@liangjingkanji
Copy link
Owner

liangjingkanji commented Mar 1, 2022 •
edited

一般场景只是聊天记录列表拉取更多记录罢了. 我不清楚你要的上下都是拉取更多记录需求要在什么场景使用

BRV是集成的SmartRefreshLayout来实现的, 你这个需求可以自己自定义刷新头RefreshHeader实现

你这种上下同时预拉取数据, 使用场景少也不好封装, 没有更好的理由BRV不考虑添加

@liangjingkanji liangjingkanji changed the title 预加载、预拉取 同时支持列表上下分页预加载 Mar 1, 2022
@liangjingkanji liangjingkanji changed the title 同时支持列表上下分页预加载 期望同时支持列表上下分页预加载 Mar 1, 2022
@CosmicRayLucky
Copy link
Author

消息搜索定位到中间位置，这个时候上下都还有未同步的数据，需要上下都进行加载。

@liangjingkanji
Copy link
Owner

liangjingkanji commented Mar 1, 2022 •
edited

这种需求不需要刷新头
你这个需求肯定是根据记录的ID去查询下一页, 无法使用BRV中的onLoadMore加载下一页.
所以我建议你使用addOnScrollListener监听滚动位置. 然后去查询到滚动倒数第几个再去请求列表数据

@CosmicRayLucky
Copy link
Author

对的，PageRefreshLayout基本是用不上的

@CosmicRayLucky
Copy link
Author

这种需求不需要刷新头 你这个需求肯定是根据记录的ID去查询下一页, 无法使用BRV中的onLoadMore加载下一页. 所以我建议你使用addOnScrollListener监听滚动位置. 然后去查询到滚动倒数第几个再去请求列表数据

老哥，你这回复真快

@liangjingkanji
Copy link
Owner

希望未来你和我一起为爱发电维护开源!

秒回是一个舔狗的基本素养 --- 郭楠

@CosmicRayLucky
Copy link
Author

CosmicRayLucky commented Mar 1, 2022 •
edited

希望未来你和我一起为爱发电维护开源!

秒回是一个舔狗的基本素养 --- 郭楠

向老哥学习，舔狗也要做狗王！

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
寻求帮助 Extra attention is needed
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@CosmicRayLucky @liangjingkanji