砖家
> @heiyexing 你在做这个feature吗。如果没有，我来做 你来吧~
这里我看了一下，之前应该是没有问题的。 `document.body.clientHeight` 是指 body 的容器高度，而不是浏览器窗口高度。  普通情况下，因为 body 会被撑起来，所以 body 自己的高度确实等于 scrollHeight。 如果我们需要 body 内部滚动，则需要给 body 定高，比如我们给 body 加上 css ```css body{ height: 100vh; } ``` 此时 `document.body.clientHeight` 高度就正确了。 ...
> 但其实，在监听对象为 `document` 的情况下，浏览器的窗口本身就可以被视之为是那个定高的容器。 body被自然撑高应该是一个比较常见的情况，我觉得可以考虑对这种情况进行支持。至少可以在文档中对hooks的使用限制进行更加明确的说明。 这里比较合理的应该是不支持 `document`，需要清晰的定义 `document.body`
你这样判断是否应该静默更新，是不是就可以 ``` const truestLoading = loading && !data ```
你现在改的太复杂了，我给个思路：在视图之外，上下必须留 1 个 buffer，所以 overscan 如果是 0，则默认等于 1。 基于这个稍微改几行代码就好了。
useCountdown uses the user's local time, maybe it should support server time, I will think about this feature.
| Name | RFC | Developer | Stage | Remark | | --- | --- | --- | ---|---| | useTimeout | #527 | @awmleer | ✅ | - |...
目前 useInfiniteScroll 是按自己的功能需求设计的 API，和 useRequest 的出入还是挺多的，所以不会透传配置。 可以单独针对 useInfiniteScroll 提一些功能诉求， useInfiniteScroll 在什么场景下需要 loadingDelay 呢？
> 有什么建议吗？ 我加下这个参数。
看了下，这里不太能直接透传 loadingDelay 参数。 因为 loading 的能力在 useInfiniteScroll 是被强依赖的，比如在滚动加载场景下，需要在 `loading=false` 时候，才会加载下一页。 如果 loading 有延迟，会导致滚动时，多次加载的问题。 这个功能需要 useInfiniteScroll 单独开发支持，而不仅仅是透传参数。