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: add touchmove event #448

Merged
merged 1 commit into from
Mar 17, 2023

Conversation

Adeams1805
Copy link
Contributor

The props draggable=false desn't work on mobile because the event is not "mousemove" but "touchmove".

I encountered the bug myself and saw this opened issue

Not a big change, just one line, but very needed in my opinion !

@Adeams1805 Adeams1805 force-pushed the feature/draggable-false-mobile branch from 5de90e2 to 20fdc56 Compare March 17, 2023 14:31
@bkrem
Copy link
Owner

bkrem commented Mar 17, 2023

Thank you for taking the time to fix this @Adeams1805 🙏

I have some catching up to do on this repo. Been swamped with professional responsibilities + out sick during the last few weeks, at the cost of any OSS contributions and addressing issues here (even this silly 1 liner bug!).

@bkrem bkrem changed the title add touchmove event fix: add touchmove event Mar 17, 2023
@bkrem bkrem merged commit 3ccee28 into bkrem:master Mar 17, 2023
@bkrem
Copy link
Owner

bkrem commented Mar 17, 2023

Fixed in https://github.com/bkrem/react-d3-tree/releases/tag/v3.5.1

Thanks again 💯

@Adeams1805
Copy link
Contributor Author

No worries ! I'm glad I was able to contribute 👍

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.

None yet

3 participants