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.

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

iOS: LazyColumn scroll bug #3572

Closed
Caij opened this issue Aug 26, 2023 · 5 comments
Closed

iOS: LazyColumn scroll bug #3572

Caij opened this issue Aug 26, 2023 · 5 comments
Labels
duplicate This issue or pull request already exists

Comments

@Caij
Copy link

Caij commented Aug 26, 2023

Describe the bug
During scrolling, LazyColumn suddenly scrolls to the top, only on the iOS platform; there are no issues on the Android platform.

Affected platforms
Select one of the platforms below:

  • All

Versions

  • Kotlin version*: 1.9.0
  • Compose Multiplatform version*: 1.4.3
  • OS version(s)* (required for Desktop and iOS issues): iPhone 8 16.4
 val model by homeComponent.model.subscribeAsState()
    val rememberLazyListState = rememberLazyListState()
    LaunchedEffect(homeComponent) {
        homeComponent.init(object : Callback<Pair<Int, Int>> {
            override suspend fun onResult(t: Pair<Int, Int>) {

            }

            override fun onError(throwable: Throwable) {

            }
        })
    }
    LazyColumn(modifier = Modifier.fillMaxSize(), contentPadding = PaddingValues(top = paddingTop),
        state = rememberLazyListState) {
        itemsIndexed(model.tweets) { index, item ->
            AllTypeCardItemContent(item)
        }
    }
RPReplay_Final1693039855.MP4
Untitled.mp4
@Caij Caij added bug Something isn't working submitted labels Aug 26, 2023
@elijah-semyonov
Copy link
Contributor

elijah-semyonov commented Aug 28, 2023

Hi, it's known and worked on currently.
Duplicate of #3335

@elijah-semyonov elijah-semyonov added duplicate This issue or pull request already exists and removed bug Something isn't working submitted labels Aug 28, 2023
@Caij
Copy link
Author

Caij commented Aug 29, 2023

@elijah-semyonov Is this fixed in version 1.5.0 release?

@elijah-semyonov
Copy link
Contributor

elijah-semyonov commented Aug 29, 2023

No, it will be fixed in a later patch after this PR is merged.

@Caij
Copy link
Author

Caij commented Sep 5, 2023

@elijah-semyonov I'm very sorry to bother you, but I would like to inquire about the timeline for addressing this issue. Many of my app users have reported this problem.

@elijah-semyonov
Copy link
Contributor

elijah-semyonov commented Sep 5, 2023

@Caij you don't bother me at all, right now the solution for this issue in PR I've linked is partly a workaround for some bugs introduced in a mechanism called synthetic events. Doing it properly without workarounds and creating regressions could take some time, because it requires some internal discussion and all targets are using this mechanism.

Compose Multiplatform for Compose is in Alpha now, so issues like that are expected and we really value your feedback and reports which helps us to solve them.

I can't give you an accurate timeline, because there are quite some moving parts, but if it's a dire issue affecting you in long term, you could build compose locally from this repo, cherry pick PR above and ship your app with locally built compose. It will be probably accompanied with other bugs introduced by us between the release and jb-main HEAD, but it's the best we can do for now.

Watch the status of this PR (and related PRs if any) and follow us on social platforms to see if it's included in the update to come or not.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
duplicate This issue or pull request already exists
Projects
None yet
Development

No branches or pull requests

2 participants