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

Merge back into react-draggable? #11

Open
STRML opened this issue Aug 4, 2015 · 3 comments
Open

Merge back into react-draggable? #11

STRML opened this issue Aug 4, 2015 · 3 comments

Comments

@STRML
Copy link

STRML commented Aug 4, 2015

Hello - I maintain react-draggable. Could you elaborate a bit on why this repository exists and what fixes you were unable to get merged?

@mikepb
Copy link
Owner

mikepb commented Aug 6, 2015

I needed bounded dragging to implement a slider UI in material-ui under a short schedule. To implemented bounded dragging, I needed to change the drag tracking algorithm and the changes grew outside of scope for simply implementing bounded dragging. react-draggable has since implemented this feature and I believe that this repo is no longer necessary.

@KapJI
Copy link

KapJI commented Sep 3, 2015

@STRML react-draggable doesn't support percentage based dragging. Also pixel based grid isn't very convenient to use. Without these features implemented it is a bit difficult to port material-ui to use react-draggable. Something like "constrain" function from this library will make your version much more flexible.

@mikepb
Copy link
Owner

mikepb commented Sep 4, 2015

Thanks for looking into it @KapJI ! It seems that using react-draggable will take a bit more effort than we thought.

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

No branches or pull requests

3 participants