turing-tech / MaterialScrollBar

An Android library that brings the Material Design 5.1 sidebar to pre-5.1 devices.
Apache License 2.0
781 stars 126 forks source link

DraggableFromAnywhere improvement #95

Closed MFlisar closed 7 years ago

MFlisar commented 7 years ago

Anything speaking against this change?

For solving this issue https://github.com/turing-tech/MaterialScrollBar/issues/57 maybe a small adjustments to care about the respondToTouch should be added as well (just the same way as the valid flag is checked now), but I'm not using this flag and not sure if it's ment for what I think it's ment for, so it's up to you to additionally add this

With this change I can make some first useable implementation in a CoordinatorLayout as well...

turing-tech commented 7 years ago

Looks good.

MFlisar commented 7 years ago

you don't need to care about the first commits, I've reverted them all because making MaterialScrollBar public destroys the need for the interface...