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

Increase tap surface for start and end handles #41

Closed
JBerendes opened this issue Mar 13, 2017 · 11 comments
Closed

Increase tap surface for start and end handles #41

JBerendes opened this issue Mar 13, 2017 · 11 comments

Comments

@JBerendes
Copy link
Contributor

The trimmer works if can tap and get ahold of the shaded area to the left of the start or to the right of the end. When it starts it's likely at 0. Obviously grabbing something with 0 width is hard. It takes about 3 to 5 tries to get ahold of the left trimmer and even more for the right one. Is there a way to increase the hitslop of these? Would anyone else find it worthwhile to add a handleHitslop property?

@JBerendes
Copy link
Contributor Author

i can help if you want to point me at a good place to start looking. If you would rather do it I can help with the pull request.

shahen94 pushed a commit that referenced this issue Mar 13, 2017
Added ability to move tracker and get that video position, also added hitSlop for left and right

corners

this push should close out #41
@shahen94
Copy link
Owner

shahen94 commented Mar 13, 2017

@JBerendes i've added 20 points hitSlop around right and left corners, so it should help
293741c#diff-6c03c8e763c79ed4239a829fdd8a7f9dR59

@JBerendes
Copy link
Contributor Author

testing now

@JBerendes
Copy link
Contributor Author

It doesn't seem to have been solved. The video starts several seconds after I have been trying to grab the starting trimmer ... Then when I'm moving it more fluidly it's because I'm grabbing the shaded area.
mar-13-2017 12-48-19

@shahen94
Copy link
Owner

Well try to change that line which I've pointed

@JBerendes
Copy link
Contributor Author

JBerendes commented Mar 13, 2017

My thought is that the area offered as touchable is not larger than what is visible. So any touch events that occur to the left of the white start trimmer are not seen because they are outside of the parent container.

I'll try what you just mentioned. Thanks

@JBerendes
Copy link
Contributor Author

JBerendes commented Mar 13, 2017

I see, we need access to a thumbWidth prop so it can be passed to ICGVideoTrimmerView.m On smaller devices it's simply to narrow at the default value.

@shahen94
Copy link
Owner

great, look how we provide all props to JS side, in similar way you can also pass thumbsWidth prop

@JBerendes
Copy link
Contributor Author

will do.

@shahen94
Copy link
Owner

@JBerendes
Copy link
Contributor Author

looks good.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants