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

fix: Need to drag draggable twice in mobile when using clones #2281

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

qiaoyuwen
Copy link

#2111
#2222
#2277

This behaviour works on 12.0.0-beta.10. So I just restore the code of bind capturing events

@atlassian-cla-bot
Copy link

atlassian-cla-bot bot commented Jul 19, 2021

Hooray! All contributors have signed the CLA.

@maxsupera
Copy link

This has been killing me for quite some time! Super excited @qiaoyuwen found the solution! Can't wait for this to be implemented 🎉

@h-dong
Copy link

h-dong commented Aug 19, 2021

Awesome stuff! I spent a whole evening looking for a workaround for this issue, would be amazing if we can get this merged as soon as possible.

@maxsupera
Copy link

maxsupera commented Aug 20, 2021

Awesome stuff! I spent a whole evening looking for a workaround for this issue, would be amazing if we can get this merged as soon as possible.

Yeah this will be great... unfortunately im stuck on 12.0.0-beta.10 since thats the only version where this bug is not present.

Also unfortunately, i think this library is barely maintained anymore, so who knows how long it will be until they integrate this fix :(

@h-dong
Copy link

h-dong commented Aug 20, 2021

Awesome stuff! I spent a whole evening looking for a workaround for this issue, would be amazing if we can get this merged as soon as possible.

Yeah this will be great... unfortunately im stuck on 12.0.0-beta.10 since thats the only version where this bug is not present.

Also unfortunately, i think this library is barely maintained anymore, so who knows how long it will be until they integrate this fix :(

Yeah, I did see the last update was 5 months ago 😞. Any issues with 12.0.0-beta.10? I might use that version if there are no obvious problems.

@maxsupera
Copy link

maxsupera commented Aug 21, 2021

Yeah, I did see the last update was 5 months ago 😞. Any issues with 12.0.0-beta.10? I might use that version if there are no obvious problems.

@h-dong I actually haven’t noticed any problems running 12.0.0-beta.10. I’m curious to know what the big differences are between v12 and v13, but for now it seems to be working fine for me. :)

@h-dong
Copy link

h-dong commented Aug 21, 2021

Yeah, I did see the last update was 5 months ago 😞. Any issues with 12.0.0-beta.10? I might use that version if there are no obvious problems.

@h-dong I actually haven’t noticed any problems running 12.0.0-beta.10. I’m curious to know what the big differences are between v12 and v13, but for now it seems to be working fine for me. :)

I'll give it a try, thanks!

@maxsupera
Copy link

maxsupera commented Sep 14, 2021

@danieldelcore Hey there! Im just tagging you here since you made the last commit to this repo and it would be great to get some attention paid to this issue so we aren't stuck on v12.0.0-beta.10!

Thanks!!

naru200 added a commit to pplink/react-beautiful-dnd that referenced this pull request Sep 30, 2021
@brienpafford
Copy link

@danieldelcore I also am tagging you, to bump this issue. When you have a chance to look at getting this fixed merged, we -- the community -- would be ever so grateful.

Thanks for all you do with this project. I don't think you get enough kudos/shoutouts as open source maintainers. We are all so thankful!

@buesing
Copy link

buesing commented Jun 13, 2023

Hey @danieldelcore just wanted to call attention to this PR again, I'm also still running into this problem. Would be so cool to get this merged!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants