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 forward seeking buttons to jwplayer #1973

Merged
merged 4 commits into from
Nov 19, 2021

Conversation

pornulus
Copy link
Contributor

@pornulus pornulus commented Nov 8, 2021

Unfortunately this feature is missing from jwplayer, but someone posted a jQuery-based workaround in the jwplayer bug tracker and I've ported it to vanilla js/typescript. (c.f. jwplayer/jwplayer#3894)

This has a little quirk: because it's mirroring the rewind button it's also mirroring the 10 which looks a little odd.
On the pro side: it adds a feature I badly want and keeps this PR simple and self contained because no additional asset needs to be added (that would possibly violate copyrights if we took the original icon and edited it)
Hopefully it's added to jwplayer upstream at some point.

Screenshot_20211108_150753
Screenshot_20211108_151126

@kermieisinthehouse kermieisinthehouse added feature Pull requests that add a new feature ui Issues related to UI labels Nov 8, 2021
@kermieisinthehouse kermieisinthehouse added this to the Version 0.12.0 milestone Nov 8, 2021
@WithoutPants WithoutPants merged commit 9a76b4c into stashapp:develop Nov 19, 2021
@kermieisinthehouse
Copy link
Collaborator

This un-centered the controls on mobile, shifting them to the top left corner. You can actually see the effect in the image in the top post.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature Pull requests that add a new feature ui Issues related to UI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants