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

Add fast forward/rewind functionality #542

Closed

Conversation

AbelTesfaye
Copy link
Contributor

Adds a customizable fast forward/rewind view that can be used for seeking the player

Bug: #132

This PR prepares the core library for testing.

It also creates a helloworld test  to be used as a template later on.
…ierfrancescoSoffritti#521)

This pull requests fixes a bug that prevented users from registering onClick/onLongCLick/onTouch listeners on YouTubePlayerView

Test: YouTubePlayerViewClickTest.kt
Bug: PierfrancescoSoffritti#105
This PR starts an emulator and runs all the tests of the core library in Travis CI.
Make web ui the default UI.

This change makes web ui the default and replaces the useWebUi attribute with useNativeUi.

Test: YoutubePlayerViewDefaultWebUITest.kt
Bug: PierfrancescoSoffritti#475
Adds a customizable fast forward/rewind view that can be used for seeking the player

Bug: PierfrancescoSoffritti#132
@AbelTesfaye
Copy link
Contributor Author

Hi,

This closes #132. But I've not included tests, because I found they would be unreliable and I need more time to figure out how to best make them for these changes. In the meantime, here is what I have so far ( proceed with caution, its not the cleanest code out there xD )

@PierfrancescoSoffritti
Copy link
Owner

Hey, the animation looks amazing, well done :) I will try to go through the code tomorrow.

@Tahsin100
Copy link

Hello @PierfrancescoSoffritti. Any update of this feature being merged?

@PierfrancescoSoffritti
Copy link
Owner

PierfrancescoSoffritti commented Dec 23, 2021

Custom UI has been removed. See 11.0.0 release notes.

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

Successfully merging this pull request may close these issues.

3 participants