I’m using the `HorizontalPager` from accompanist l...

# compose

[nglauber]

I’m using the HorizontalPager from accompanist library. In version 0.9.0 the offscreenLimit parameter was removed, but it’s still in the docs…
Is there any replacement for this param? cc: @cb

[cb]

We probably should have made that clearer. It’s now set on the pager state

[nglauber]

Oh! you mean this? 👇

rememberPagerState(
    pageCount = images.size,
    initialOffscreenLimit = 2
)

[nitrog42]

btw I have a bug where when we scroll to the last item, any scroll movement to the previous item will be "instant", without any animation/scroll. You touch the pager and it's instantly to the element before. It happens only for the last item though. I'll file a bug report but I want to isolate it in a reproducible sample project first. Do you have the same behavior ?

[nglauber]

😮 @nitrog42 I just noticed that

[nitrog42]

good so I'm not the only one !

[nglauber]

There’s a ticket to that…
https://github.com/google/accompanist/issues/395

[nitrog42]

oh well I think the original ticket isn't about the same issue, but the comment and the PR linked is definitelly what happens for us so it should be fixed soon!

[cb]

The attached PR fixes that issue too. I’ll get it merged and a new release out tonight or tomorrow.

[nglauber]

Thanks @cb! and congrats for you great job 😉

[nglauber]

hey @cb. could you please take a look into this?
https://stackoverflow.com/questions/67419418/how-to-add-swipe-behavior-to-the-screens/67420805
The bug we mentioned here in slack is fixed, but now I’m getting a different one…

[cb]

What’s the new issue? I don’t see anything new there 🤔

[nglauber]

When I scroll the last page back, the bottom tab is not updated. Usually from the last tab, but it also happened in the third tab.

[nglauber]

[cb]

And that is with 0.9.1?

[cb]

We added a test which does exactly what you're doing in that demo (which is passing). Looks like a more nuanced issue somewhere

https://github.com/google/accompanist/blob/main/pager/src/androidTest/java/com/google/accompanist/pager/PagerTest.kt#L86

[nglauber]

Yes… 0.9.1. 😞

[cb]

Found the issue, I won't be able to do another release though until next week

[cb]

(you can use the snapshots if needed though)

[nglauber]

Great! 👏
No worries 😉 I’m using in a playground repo. No rush

[cb]

https://github.com/google/accompanist/pull/400

[nglauber]

Awesome! Thanks again 😊

[allan.conda]

offscreenLimit has been completely removed now. @nglauber Have you found a solution to implement an offscreenLimit ? 🙏

[nitrog42]

well you shouldn't need it anymore, as now the pager use a lazy mechanism

[allan.conda]

Well now the adjacent images do not load until I swipe to the item.

[Paul]

@allan.conda Have you found how to do it? I'm facing a similar issue.

[allan.conda]

Pager’s implementation is LazyColumn now, so I think it’s not possible until that API supports preloading.
You can star my dead request 😄 https://issuetracker.google.com/issues/172029355