Skip to content
This repository has been archived by the owner on Nov 22, 2017. It is now read-only.

Draggable view is not working as expected in minimized state when moving left or right. #20

Closed
manpreetsinghsodhi opened this issue Nov 11, 2014 · 11 comments
Assignees
Labels

Comments

@manpreetsinghsodhi
Copy link

Hi, when i drag view in left or right in minimized form then vedio is not behaving or moving properly.
I tried to do some changes in viewcallback class but not working my way.
It is not going smooth for me i.e on dragging right the view is going completely left.

I checked your app on play store, there it is working fine but when i run your sample app i am facing these issues.

Moreover when view is dragged to minimized state then it got some margins in left and right in every vedio.
Please help me resolve this issue.
Thanks

@pedrovgs
Copy link
Owner

Can you upload any screenshot or video to show this problem? I'm not able to reproduce it.

@manpreetsinghsodhi
Copy link
Author

Sure. Here is the link of video i am facing problem http://tinypic.com/player.php?v=289wjo7%3E&s=8#.VGLjdMluVlM
Issues with this video when dragged left and right on minimized state are :

  1. On dragging right halfway, it is going left and vica versa.
  2. And when video is dragged out(via left or right), it comes back again in minimized state(on right side) sometimes.
  3. Fading effect(alpha) while going left and right fades video completely as soon as we drag. Fading should be such that video keep on playing and should fade lightly then completely at corner.
    And lastly one minor issue is dragging horizontally was not that smooth . Vertically it was very very smooth.

@pedrovgs pedrovgs added the bug label Nov 30, 2014
@pedrovgs
Copy link
Owner

@manpreetsinghsodhi can you take a look at https://github.com/pedrovgs/DraggablePanel/tree/fix-horizontal-drag-problems I've been working to improve horizontal drag implementation and it seems to work again in this branch :)

Once you validate this branch I will release this feature to 1.4 version

@manpreetsinghsodhi
Copy link
Author

Hi Pedrovgs,
First of all thanks for taking time in solving the issue.
I tested this branch. It is working fine on left drag(on minimized view) but when we drag right sometimes it goes left again(out of the screen) and whenever this particular case happens the view after going out of screen comes back in minimized state on right side . Rest everything is fine..

@pedrovgs
Copy link
Owner

pedrovgs commented Dec 1, 2014

Can you upload a video with that movement? Thanks!
On Dec 1, 2014 9:30 AM, "manpreetsinghsodhi" notifications@github.com
wrote:

Hi Pedrovgs,
First of all thanks for taking time in solving the issue.
I tested this branch. It is working fine on left drag(on minimized view)
but when we drag right sometimes it goes left again and whenever this
particular case happens the view after going out of screen comes back in
minimized state on right side . Rest everything is fine..


Reply to this email directly or view it on GitHub
#20 (comment)
.

@manpreetsinghsodhi
Copy link
Author

Here is the link of video . Please have a look. Thanks !
http://tinypic.com/player.php?v=zwjn7q%3E&s=8#.VH1UZ8ladWE

@Fiddl3
Copy link
Contributor

Fiddl3 commented Dec 2, 2014

in triggerOnReleaseActionsWhileVerticalDrag() method of DraggableViewCallback:

edit:
sorry in triggerOnReleaseActionsWhileHorizontalDrag()

...

       if (xVel < 0 && xVel <= -X_MIN_VELOCITY) {//the mising "-" before (Y_MIN_VELOCITY) caused this bug
            draggableView.closeToLeft();
        }

...

@pedrovgs
Copy link
Owner

pedrovgs commented Dec 2, 2014

Thanks to @Fiddl3 I've found the bug \o/. @manpreetsinghsodhi can you review if it's working fine for you?

@pedrovgs pedrovgs self-assigned this Dec 3, 2014
@pedrovgs
Copy link
Owner

pedrovgs commented Dec 4, 2014

@manpreetsinghsodhi can you review if this branch is working fine for you and fixes the bug related to the horizontal drag? I'd like to integrate this branch and release a new version :)

Fiddl3 added a commit to Fiddl3/DraggablePanel that referenced this issue Dec 5, 2014
Fixed SecondView positionig in Maximized State
Fixed Preview in Android Studio (DraggablePanel should be treated as
RelativeLayout) ex: layout\activity_tv_shows_sample.xml
Fixed horizontal drag issues
@manpreetsinghsodhi
Copy link
Author

Hi ,yes it is working fine now . Thanks @pedrovgs and fiddle :)

@pedrovgs
Copy link
Owner

pedrovgs commented Dec 5, 2014

\o/ @manpreetsinghsodhi I'll merge that branch and generate a new release asap. Thanks for your help!

@pedrovgs pedrovgs closed this as completed Dec 5, 2014
pedrovgs added a commit that referenced this issue Dec 5, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

3 participants