Merge back into react-draggable?

mikepb / react-draggable

React draggable component

20 stars 6 forks source link

Merge back into react-draggable? #11

Open STRML opened 9 years ago

STRML commented 9 years ago

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 commented 9 years ago

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 commented 9 years ago

@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 commented 9 years ago

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