I’m experiencing an issue where scrolling on a cli...

# compose

[dambakk]

I’m experiencing an issue where scrolling on a clickable composable (in this case a Card but it happens on buttons as well) makes it non-clickable the next time you try to click it, and after that it is clickable again. I guess it’s somehow related to the scroll+ripple bugfix included in beta08, but not what I want here. The card is wrapped in a Column that is not scrollable if that matters, and it’s only the banner that is written in Compose. Is there anything I can do?

screen-20210622-132002.mp4

[Andrey Kulikov]

@matvei do you have any ideas?

[matvei]

Hm, not sure, but it looks like we don't properly cleanup if cancelled? idk. cc @Louis Pullen-Freilich [G] for the ripple stuff that might be related

[matvei]

Just tested a small local demo with clikcable item with ripple in the lazy column and it works fine. You you please post your code here as well, Christopher?

[matvei]

Also, it seems like you are in beta08? Could you please try beta09 as there was one more fix for ripples in there

[dambakk]

Yeah, I just upgraded to beta09 and can confirm that issue is still there.
So Im using a ComposeView with this content:

Theme
  Column
    Text
    Box
      Card <- this has onClick
        Image
      Text
      ..

Do you need the full code, or are you able to reproduce with this?

[matvei]

Oh so the scrolling is done with RecyclerView, not LazyColumn?

[dambakk]

No, I have a NestedScrollView, but yes, I’m not using LazyColumn. Its only the red banner that is written in Compose 🙂

[matvei]

Ok, so if you will have some time to come up with the small repro (similar to one that you listed above, but with NestedScrollView as well and compilable) -- that would greatly help us debug it further

[dambakk]

Yeah, of course! I’ll get right to it!

[dambakk]

There we go! https://github.com/Dambakk/ComposeScrollClickIssue Do you need anything else? Let me know if I can help with anything.

[dambakk]

screen-20210622-163533.mp4

[matvei]

No, this is everything we need, thanks a lot for making a repro. I've filed https://issuetracker.google.com/issues/191758125 and we will investigate further!

[dambakk]

No problem! And thanks for the quick reply!

[dambakk]

Btw, @matvei, do you think this issue is in scope to be fixed before compose hits stable?

[matvei]

I don't have a good answer for this yet, sorry 🙂 We're still yet to tackle this, so the complexity is unknown, stay tuned

[dambakk]

Wow, merged already? 😮 Thats impressive, well done you guys! 👏 Does that mean it will be available in the next release, or do you do some sort of scheduling for these things?

[matvei]

Should be available in the future rc02 🙂