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

Using LazyColumn.contentPadding shifts draggable position #39

Closed
alashow opened this issue Nov 1, 2021 · 6 comments · Fixed by #40
Closed

Using LazyColumn.contentPadding shifts draggable position #39

alashow opened this issue Nov 1, 2021 · 6 comments · Fixed by #40
Assignees
Labels
bug Something isn't working

Comments

@alashow
Copy link

alashow commented Nov 1, 2021

If contentPadding used on a LazyColumn (to set status bar padding & navigation bar padding for example), draggable items's positions are shifted.

So if draggable items height is 32.dp, and LazyColumn(contentPadding=PaddingValues(top=32.dp)) is used, trying to drag the first item actually drags the second, or whatever item is next.

@aclassen
Copy link
Owner

aclassen commented Nov 1, 2021

I can't reproduce this issue using the sample app. Maybe you can share your code.

@alashow
Copy link
Author

alashow commented Nov 1, 2021

I was able to reproduce this by changing the sample app: alashow@e32653e . If you have already tried to reproduce this way, maybe it's device/os specific issue? I tested on Android 12 emulator & pixel 4a.

@aclassen aclassen self-assigned this Nov 2, 2021
@aclassen aclassen added the bug Something isn't working label Nov 2, 2021
@aclassen aclassen linked a pull request Nov 2, 2021 that will close this issue
@aclassen aclassen closed this as completed Nov 2, 2021
@alashow
Copy link
Author

alashow commented Sep 19, 2022

Seems like this issue has regressed. It's happening again in 0.9.2.

@Thomas-Vos
Copy link

Can also confirm the issue is back. Could this be fixed?

@myounisWajeez
Copy link

I'm also facing the same issue on 0.9.6

@agcollins
Copy link

This is still an issue. Please reopen.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants