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

Double tap to forward & Backward #3

Closed
ghost opened this issue Dec 19, 2020 · 5 comments
Closed

Double tap to forward & Backward #3

ghost opened this issue Dec 19, 2020 · 5 comments
Labels
enhancement New feature or request

Comments

@ghost
Copy link

ghost commented Dec 19, 2020

Add option to double tap to forward & Backward (10 second )
huyfyfy_1

@moneytoo
Copy link
Owner

I thought about this one but I wasn't sure. Is it really needed since we already have the swipe to seek? Sure, it can be done but do other players also provide both ways? I would like to keep it clean and simple. The double tap gesture could be potentially used for some other action.

@ghost
Copy link
Author

ghost commented Dec 19, 2020

Pls Dev, It is useful feature it not affect for simplistic.

@moneytoo moneytoo added the enhancement New feature or request label Dec 19, 2020
@deepu2
Copy link

deepu2 commented Dec 26, 2020

Double tap is simple at times when you just want to skip 10s without fiddling with the swipe. Also it's more intuitive now that it's been implemented everywhere.

@AuthenticPumpkin
Copy link

Hey Dev. I agree with the feature request, came here to make that myself and saw this one. I get your point about keeping it simple. To me slide to seek and double tap to seek are 2 very different features for different use cases. Slide to seek is very inconvenient for the most commonly used seeking - going back or forward 5 or 10 seconds to quickly review something you missed. Double tap works great for that and is very common in other players, thus very intuitive and expected everywhere from my perspective.

Please do consider adding this to your lovely app! Best video player with a target SDK of 30 in my experience, thank you :)

@moneytoo
Copy link
Owner

Added in latest pre-release

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants